Results for 2.97 20010212 (experimental) testsuite on i686-pc -cyg win

[Christopher Faylor]

On Wed, Feb 28, 2001 at 07:48:33AM +0000, Neil Booth wrote:
>DJ,
>
>Does this look OK to you?  If so, OK to commit for branch and mainline?

I've been working in this section of the code.  I missed the earlier
discussion about this -- I guess I should start reading gcc-bugs, maybe?

Anyway, I don't know what "this problem" is.  Could you elucidate?

The intent of the code you are changing was to duplicate the normal search
order that I thought gcc used.   I thought the -isystem ... -idirafter
would cause /usr/local/include to be searched before any target specific
includes and /usr/include to be searched after the target specific includes.
If it is incorrect I'd like to know why.

cgf

>Billinghurst, David (CRTS) wrote:-
>
>> I have a solution to this problem.  It may even be correct.
>> 
>> The cause of the problem is using "-idirafter /usr/include" rather than
>> "-isystem /usr/include".  This is fixed with the following patch, which
>> bootstraps on the gcc-3.0 branch and eliminates the cpp testsuite failures.
>> 
>> --- gcc/config/i386/cygwin.h.orig	Tue Feb  6 21:11:08 2001
>> +++ gcc/config/i386/cygwin.h	Tue Feb 27 20:08:33 2001
>> @@ -88,7 +88,7 @@
>>  #define MINGW_LIBS "-L" CYGWIN_CROSS_DIR "/lib/mingw"
>>  #define MINGW_INCLUDES "-I" CYGWIN_CROSS_DIR "/include/mingw"
>>  #else
>> -#define CYGWIN_INCLUDES "-isystem /usr/local/include -idirafter
>> /usr/include"
>> +#define CYGWIN_INCLUDES "-isystem /usr/local/include -isystem /usr/include"
>>  #define CYGWIN_W32API "-I/usr/include/w32api"
>>  #define CYGWIN_LIB "/usr/lib"
>>  #define MINGW_LIBS "-L/usr/local/lib/mingw -L/usr/lib/mingw"
>> 
>> I don't understand why -idirafter was used.  There are only a few
>> occurrences of it in the source tree. If it wrong for cygwin it may be wrong
>> elsewhere as well.
>> 
>> Your thoughts?