Dejagnu: use -isystem to include system header files.
Nick Clifton
nickc@redhat.com
Thu Nov 11 17:00:00 GMT 2004
Hi Daniel,
> This patch will break in-tree testing for yet other targets. I believe
> arm-elf was affected - anything which does not set
> NO_IMPLICIT_EXTERN_C. I discussed this with H-P on the dejagnu list
> but never figured out a solution, but...
>> * lib/libgloss.exp (newlib_include_flags): Use -isystem, not -I.
>> (libio_include_flags, g++_include_flags, libstdc++_include_flags,
>> winsup_include_flags): Ditto.
> ... I strongly suspect that g++ and winsup should be left out.
Ok - what about this revision to the patched sources based upon Rob
Savoye's suggestion that -isystem is only needed for newlib includes ?
It appears to work for the xstormy16 port and I did not detect any
regression for the arm-elf port, so I think that it should be OK.
Assuming that you like this patch, shall I submit it to Rob for
inclusion in the official sources as well ?
Cheers
Nick
dejagnu/ChangeLog
2004-11-11 Nick Clifton <nickc@redhat.com>
* lib/libgloss.exp (libio_include_flags, g++_include_flags,
winsup_include_flags): Revert previous patch, restoring the use of
-I, for all libraries except newlib. Newlib needs -isystem to
avoid the problems with <limits.h> but the C++ and winsup
libraries need -I because -isystem generates an implicit 'extern
"C"' which may not be appropriate for certain targets.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: libgloss.exp.patch
URL: <http://sourceware.org/pipermail/newlib/attachments/20041111/707ceb0c/attachment.ksh>
More information about the Newlib
mailing list