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 -