This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Joseph S. Myers - fixinc v. glibc <values.h>


> I think from the comment, we can add:
> 
>   test =
>   "-n \"`egrep '#[ \t]*define[ \t]+MAXINT[ \t]' sys/param.h`\"";
> 
> which will cause the fix to be applied only if the egrep
> finds matching text in sys/param.h.  (The fixinclude working
> directory is /usr/include.)

I was the one that added values.h to this test as the original fix only
worked if sys/param.h was included before values.h.  In my original patch,
I had a machine selector which which was removed on Jeff Law's suggestion.
Anyway, I have no objection to the above.  However, a while back I submitted
this patch <http://gcc.gnu.org/ml/gcc-patches/2000-10/msg00618.html>
to prevent application under hpux 11 which already has the MAXINT defines
wrapped in both sys/param.h and values.h.

Dave
-- 
J. David Anglin                                  dave.anglin@nrc.ca
National Research Council of Canada              (613) 990-0752 (FAX: 952-6605)

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]