This is the mail archive of the
mailing list for the binutils project.
Re: Dejagnu: use -isystem to include system header files.
- From: Daniel Jacobowitz <drow at false dot org>
- To: Nick Clifton <nickc at redhat dot com>
- Cc: binutils at sources dot redhat dot com, gdb-patches at sources dot redhat dot com,newlib at sources dot redhat dot com
- Date: Thu, 11 Nov 2004 09:22:37 -0500
- Subject: Re: Dejagnu: use -isystem to include system header files.
- References: <email@example.com>
On Thu, Nov 11, 2004 at 11:58:15AM +0000, Nick Clifton wrote:
> Hi Guys,
> I am going to check in the attached patch which imports a fix from
> the mainline dejagnu sources. This fix is to use the -isystem
> switch to include system header files rather than -I. This fixes
> several unexpected failures in the GCC and G++ testsuites where the
> newlib system header file <limits.h> is included in strict ANSI
> mode, and the compiler barfs on the #include_next directive.
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.