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:subject:to:references:from:message-id:date
	:mime-version:in-reply-to:content-type; q=dns; s=default; b=ezvb
	2rt1a529yNDQLkipgrwAvvigEIuPdPUUu34S1M9zcj7qZF8AGtYHr+PorAI+9vTd
	2SRIEE3A8yxyLfbQJU8WcICCjKxFBn6JifOlyFlKp6KGajt0oVhv6zbjAGbBsp3I
	iRauNBNCKCg2Q8NiG+0ZmIgkRKzg0cil3SqpvMw=
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:subject:to:references:from:message-id:date
	:mime-version:in-reply-to:content-type; s=default; bh=TpWBVi1/dC
	4KMhziqm+lCIQOMDw=; b=B8ZEb1oj+tBFTuVJEwPZHrpTrubiKh3RIMlc4z4r2/
	ZuKB1XH11MRsZkb5blZJ8ZzIpJGEBMyQo4ngsWLXrSh2JPmGEuT2qu1JgABoHMXs
	1oTtjnd0phrQkjWMx894LV7LMrudNyfSgdY71h9O8HOTiVewSgwwfkJjH7nfHo9e
	8=
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=-1.8 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=kick, virtualization, OpenBSD, H*f:sk:56731D8
X-HELO:	mx1.redhat.com
Subject:	Re: [ANNOUNCEMENT] TEST RELEASE: Cygwin 2.4.0-0.11
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>
From:	Eric Blake <eblake AT redhat DOT com>
Openpgp:	url=http://people.redhat.com/eblake/eblake.gpg
Message-ID:	<56732621.6040101@redhat.com>
Date:	Thu, 17 Dec 2015 14:16:17 -0700
User-Agent:	Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0
MIME-Version:	1.0
In-Reply-To:	<20151217210107.GC3507@calimero.vinschen.de>
X-IsSubscribed:	yes

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

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/selec=
t.h
>> because of the recent changes.  This brings in Gnulib's sys/select.h, wh=
ich
>> includes signal.h.  We then get the errors I posted because we haven't y=
et
>> 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 rea=
lly
>> belong in sys/types.h.  So why does the latter include sys/select.h?
>=20
> Because it's done exactly the same way on FreeBSD and OpenBSD:
>=20
>   # if    __BSD_VISIBLE
>   #include <sys/select.h>
>   [...]
>=20
> Gnulib should allow to work with this to be portable.  So why does gnulib
> provide its own sys/select.h?  Is it configurable?

I'll raise this on the gnulib list, and we can come up with workarounds.
 But bypassing gnulib's sys/select.h probably won't work in general
(gnulib tends to always provide wrapper headers that use #include_next
to the system header, even if nothing is replaced, because it is easier
than trying to figure out when a wrapper header is needed).  Of course,
fixing it in gnulib won't help the myriad of projects that will be
bitten by this until they do a new release based on updated gnulib, so
maybe we can also find a way to make cygwin play nice without a gnulib
update required.  I'm still investigating.

--=20
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


--c1ho4MVcRIPfqeLFtac4WaFjm5jjDH6QA
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2
Comment: Public key at http://people.redhat.com/eblake/eblake.gpg
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBCAAGBQJWcyYhAAoJEKeha0olJ0Nq3EUIAK+Lu+aiC6DsMieHrbsMIeXF
2lsyL5A+Lb9HrouSIQAw10hnzJ+xb6+ucqRoOnvev4Mxejtgte5/0L30Msk5EFSU
VI//2LPruu+zxMjWqwQVPBBAH5nFwwrZhg6CBLBDQUnTJUKefth5Fry1NO1bKCBo
Uxyc8SXuEBRucoFd5V/vDPkg6SSUCJtR3aF/lZqD41iFRMdIlypGwPcXehlrb8q8
GhS4rOE2jaIPkFmfwwm4cShXzgvJ0DYziGOYKfd58qIUtgcWvJjxmLzROsBVnaSG
PEsu3WlFNm4327Xyo0hLStRxW46S3a3HufB8S8gC08RQ/qRM+uS2Le9nwMAF050=
=qtXS
-----END PGP SIGNATURE-----

--c1ho4MVcRIPfqeLFtac4WaFjm5jjDH6QA--

- Raw text -