This is the mail archive of the cygwin 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]
Other format: [Raw text]

Re: [ANNOUNCEMENT] TEST RELEASE: Cygwin 2.4.0-0.11

On Dec 17 14:40, Eric Blake wrote:
> On 12/17/2015 02:16 PM, Eric Blake wrote:
> > On 12/17/2015 02:01 PM, Corinna Vinschen wrote:
> > 
> >>> Here's what happens:
> >>>
> >>> One of the Gnulib modules includes sys/types.h, which includes sys/select.h
> >>> because of the recent changes.  This brings in Gnulib's sys/select.h, which
> >>> includes signal.h.  We then get the errors I posted because we haven't yet
> >>> finished including sys/types.h.
> > 
> > Gnulib has been taught to work around early inclusion problems before;
> > sounds like this will be another case where gnulib has to make sure the
> > system header is complete before its own replacements kick in.
> > 
> >>>
> >>> All the build errors disappear if I remove '#include <sys/select.h>' from
> >>> sys/types.h.  You said above that the macros related to select don't really
> >>> belong in sys/types.h.  So why does the latter include sys/select.h?
> Another data point: POSIX does NOT allow <sys/types.h> to pollute the
> namespace with symbols from <sys/select.h>.  True, the use of
> __BSD_VISIBLE says that POSIX is not in play, but I'm suspecting that
> very few programs are written that use sys/select.h functionality but
> were relying on the BSD headers to indirectly include it via
> sys/types.h, since such programs would fail to compile on other systems
> where the indirect include is not present (more likely, any clients of
> sys/select.h are directly including it).
> So at this point, I'm leaning towards fixing the cygwin header to not
> include sys/select.h from sys/types.h.

I agree.  I reverted this part of the patch, despite being compatible
with some of the BSDs.  It very obviously breaks backward compat.

I'm just building new developer snapshots (give them half an hour) and
I'll upload a new cygwin test release probably tomorrow.


Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

Attachment: pgpYyxSGHDhJE.pgp
Description: PGP signature

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