From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28743 invoked by alias); 17 Nov 2004 17:47:14 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 27965 invoked from network); 17 Nov 2004 17:46:41 -0000 Received: from unknown (HELO cam-admin0.cambridge.arm.com) (193.131.176.58) by sourceware.org with SMTP; 17 Nov 2004 17:46:41 -0000 Received: from pc960.cambridge.arm.com (pc960.cambridge.arm.com [10.1.205.4]) by cam-admin0.cambridge.arm.com (8.12.10/8.12.10) with ESMTP id iAHHjqso029854; Wed, 17 Nov 2004 17:45:52 GMT Received: from pc960.cambridge.arm.com (localhost.localdomain [127.0.0.1]) by pc960.cambridge.arm.com (8.12.8/8.12.8) with ESMTP id iAHHkYgK023271; Wed, 17 Nov 2004 17:46:34 GMT Received: (from rearnsha@localhost) by pc960.cambridge.arm.com (8.12.8/8.12.8/Submit) id iAHHkQWJ023269; Wed, 17 Nov 2004 17:46:26 GMT X-Authentication-Warning: pc960.cambridge.arm.com: rearnsha set sender to rearnsha@gcc.gnu.org using -f Subject: Re: Dejagnu: use -isystem to include system header files. From: Richard Earnshaw To: nickc@redhat.com Cc: binutils@sources.redhat.com, gdb-patches@sources.redhat.com, newlib@sources.redhat.com Content-Type: text/plain Content-Transfer-Encoding: 7bit Organization: GNU Message-Id: <1100713585.23221.6.camel@pc960.cambridge.arm.com> Mime-Version: 1.0 Date: Wed, 17 Nov 2004 17:47:00 -0000 X-SW-Source: 2004-11/txt/msg00358.txt.bz2 Nick Clifton wrote: 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 is included in strict ANSI mode, and the compiler barfs on the #include_next directive. Unfortunately this patch causes regressions on the gcc builtins tests. These tests rely on detecting newlib by looking for the definition of _NEWLIB_VERSION being added by including limits.h; but the change in the search order means that we now pick up a dummy version of newlib.h from the gcc include directory. With your patch the search path has now become /work/rearnsha/gnu/egcs/gcc/include /work/rearnsha/gnu/egcs/arm-elf/./newlib/targ-include /home/rearnsha/gnusrc/egcs-cross/newlib/libc/include Whereas previously the gcc/include directory came later in the search. R.