This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [Bug-dejagnu] [RFA:] DejaGNU patch: Change -I to -isystem for testsuite.
- From: Rob Savoye <rob at welcomehome dot org>
- To: Hans-Peter Nilsson <hp at bitrange dot com>
- Cc: bug-dejagnu at gnu dot org, gdb-patches at sources dot redhat dot com
- Date: Tue, 12 Nov 2002 13:35:32 -0700
- Subject: Re: [Bug-dejagnu] [RFA:] DejaGNU patch: Change -I to -isystem for testsuite.
- References: <Pine.BSF.4.44.0210262330040.27841-200000@dair.pair.com>
On Sat, Oct 26, 2002 at 11:33:41PM -0400, Hans-Peter Nilsson wrote:
> system_header", it shouldn't be needed; it'd be better if gcc
> treated it as a system header while running the test-suite.
> Done by changing -I to -isystem for include directives for the
> test-suite. The -isystem option has been there since at least
> gcc-2.7.2 so it should be safe for all uses. This patch is
Makes sense.
> BTW, does anybody have an idea why gcc/testsuite/site.exp is
> completely ineffective? Changing s/-I/-isystem/g in the
> site.exp in gcc/Makefile.in has no effect; in fact it doesn't
> seem to be used at all.
I believe the site.exp that is actually read in is from the
gcc directory, not the gcc/testsuite one.
> Ok to commit to sources.redhat.com/dejagnu?
Looks ok to me. I'll check this into savannah after I merge in the patch.
- rob -