This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: cygwin/newlib types patchs


On Wed, Mar 21, 2001 at 09:05:59AM -0500, Christopher Faylor wrote:
>On Wed, Mar 21, 2001 at 08:10:00PM +1100, Robert Collins wrote:
>>This diff was taken from the src/ level (immediately above winsup and
>>newlib) - but I've split it into two... however I'm mailing them
>>together because if only one is applied, cygwin & newlib will no longer
>>place nicely together.
>>
>>Chris, I hope this is a bit easier on you...
>>
>>
>>newlib ChangeLog:
>>21 Mar 2001 Robert Collins <rbtcollins@hotmail.com>
>>    * libc/include/sys/signal.h: Test for __CYGWIN__ as well as
>>_POSIX_THREADS
>>    * libc/include/sys/types.h:
>>      Allow __CYGWIN__ for the POSIX_THREADS types.
>>      Include <cygwin/types.h> rather than use the _POSIX_THREADS
>>structs from newlib.
>
>This is, again, odd ChangeLog formatting.  The wrapping is weird
>and the indentation is off.
>
>Anyway, shouldn't we be turning on _POSIX_THREADS in
>newlib/libc/include/sys/features.h ?  It doesn't seem right to
>be checking explicitly for both a generic case (_POSIX_THREADS)
>and a system specific case (__CYGWIN__).  The _POSIX_THREADS
>definition is there to control this, isn't it?

Or, looking at your other comments, maybe we just need finer grained
control of what features are defined in features.h for Cygwin.

cgf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]