X-Recipient:	archive-cygwin AT delorie DOT com
DomainKey-Signature:	a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; q=dns; s=
	default; b=uQbPGIDHgTD2UNmwAi6mKPJm8H7lQST21Qnn0kUi5uzwXPVP7kMdE
	eqnnMrnkQcV4QwagIzPYrWvrMC0C5iJeHXxBvbWywFd+3mvE4wdRQ9OF1w/6I0RO
	c9+1Eisp/FvUms5pvzhHb5cIg30X0Tt10Hu8bHbBCT+ZdDboFtI3EE=
DKIM-Signature:	v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; s=default;
	bh=npfJGtG4RfUcODpKai8qWFKjOgo=; b=y7hmTMjMZxrE7rkusaOQjCmN0vxP
	PAqeKrq9IpGFOKQebQLoQvzbLe8kmK/ZzicHK8F49I0h8krl/tdMod6FLR94jFGH
	s7dxDF+LYKcD4IiK8t3H38G/N4AMlg4EhzTFuJFKw9Dxb+1+jTwd91OI5xL2SBh3
	jNiRTxoZCikVpsY=
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com
Authentication-Results:	sourceware.org; auth=none
X-Virus-Found:	No
X-Spam-SWARE-Status:	No, score=-101.9 required=5.0 tests=AWL,BAYES_50,KAM_LAZY_DOMAIN_SECURITY,RCVD_IN_PBL,RDNS_DYNAMIC,USER_IN_WHITELIST autolearn=no version=3.3.2 spammy=HX-HELO:sk:calimer, H*r:188.192.47, Hx-spam-relays-external:sk:ipbcc02, Hx-spam-relays-external:188.192.47.232
X-HELO:	calimero.vinschen.de
Date:	Thu, 17 Dec 2015 22:56:41 +0100
From:	Corinna Vinschen <corinna-cygwin AT cygwin DOT com>
To:	cygwin AT cygwin DOT com
Subject:	Re: [ANNOUNCEMENT] TEST RELEASE: Cygwin 2.4.0-0.11
Message-ID:	<20151217215641.GE3507@calimero.vinschen.de>
Reply-To:	cygwin AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
References:	<20151216164848 DOT GM3507 AT calimero DOT vinschen DOT de> <5671EFF8 DOT 6030804 AT cornell DOT edu> <20151217093634 DOT GP3507 AT calimero DOT vinschen DOT de> <56730350 DOT 1080002 AT cornell DOT edu> <20151217201709 DOT GA28305 AT calimero DOT vinschen DOT de> <56731D8C DOT 9020001 AT cornell DOT edu> <20151217210107 DOT GC3507 AT calimero DOT vinschen DOT de> <56732621 DOT 6040101 AT redhat DOT com> <56732BEB DOT 7020507 AT redhat DOT com>
MIME-Version:	1.0
In-Reply-To:	<56732BEB.7020507@redhat.com>
User-Agent:	Mutt/1.5.23 (2014-03-12)

--eNjIDde0W37E3OQP
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

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:
> >=20
> >>> Here's what happens:
> >>>
> >>> One of the Gnulib modules includes sys/types.h, which includes sys/se=
lect.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.
> >=20
> > 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.
> >=20
> >>>
> >>> 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?
>=20
> 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).
>=20
> 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.


Thanks,
Corinna

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

--eNjIDde0W37E3OQP
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBCAAGBQJWcy+ZAAoJEPU2Bp2uRE+gZSYP/1yvf9MoF0Xrg5sdKmp4Odlx
9nYQ9gdYP3Kji3ha+78Qm+c4I0pkIieyA94mdNBhHzyOnwHeXKhZZf9fhocpRcsE
kET/Dwp3cHb8eKKhYQYNMu7vSKV83AaGrtXQlf+BLeCavTLJnSDQOGK3gcNp53Wv
bqHZNsq2PRI0Rn33+BzWkLN9SdoEcuEsOzD4dcikbimUbCVdQpZoUQmexGnwtpAZ
nbUSmoQCGD4EQ6NdINzjFIkFRA0ibdOb9a7buIY+tl7WjENM2PmHUvOIHNlkUfZO
C7FMFmRreNgQD+sHSDCtHtmtcxvS6cm4AZXl3u7I8aJQZQ6oqkmuqsKuGz06PfcM
oCXPCZqRynWbJDp7BZsWTDC4V3sfdUleYdaIIB+EyuPdPrbVuAVcslq5ggLWJqDV
Nx+2ZVYcbx4VKzk1+kLADoneo6idNTVEFWJe7PkU0NauFyUyojWdagO4frVoGCGY
Pa6aJiC4DqWFFrdAxfv+8i4Q4odmAkc6TFOD7uC+FylkVi3LYu1kFd4zK/22VGTE
jTXKvD0NB4Rj7J72FL/PiRpubr1bFxfty5G1gKZJ80P6VvRPJbHlxjERBaGRcMaX
u8n3UPoUBfTVGUbVbbw8OcTYrohzPYvrh46VbO/uUtRvJ7VGrwm3TCKw+CaHqfmD
pJowAv6yP448rvWONAwl
=vRmC
-----END PGP SIGNATURE-----

--eNjIDde0W37E3OQP--

- Raw text -