Re: [Bug-dejagnu] [RFA:] DejaGNU patch: Change -I to -isystem for testsuite.

Rob Savoye <[email protected]>
Newsgroups gmane.comp.gdb.patches,gmane.comp.sysutils.dejagnu.bugs
Message-ID <[email protected]>
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 -
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.