Re: [PATCH 1/1] cygwin: use CREATE_DEFAULT_ERROR_MODE in spawn

2020-12-08 Thread Jeremy Drake via Cygwin-patches
On Tue, 8 Dec 2020, Jon Turney wrote: > On 07/12/2020 09:43, Corinna Vinschen wrote: > > On Dec 4 10:35, Jeremy Drake via Cygwin-patches wrote: > > > On Fri, 4 Dec 2020, Corinna Vinschen via Cygwin-patches wrote: > > > > > > > I'm not happy about a new CYGWIN option. > > > > > > > > Wouldn't it

Re: [PATCH] Use automake (v3)

2020-12-08 Thread Jon Turney
On 02/12/2020 19:03, Corinna Vinschen via Cygwin-patches wrote: On Dec 2 13:33, Yaakov Selkowitz via Cygwin-patches wrote: On Wed, 2020-12-02 at 18:03 +, Jon Turney wrote: On 02/12/2020 17:05, Corinna Vinschen via Cygwin-patches wrote: On Dec 2 15:36, Jon Turney wrote: On 01/12/2020

Re: [PATCH] Cygwin: Launch cygmagic with bash, not sh

2020-12-08 Thread Jon Turney
On 07/12/2020 06:17, Mark Geisert wrote: On some systems /bin/sh is not /bin/bash and cygmagic has bash-isms in it. So even though cygmagic has a /bin/bash shebang, it also needs to be launched with bash from within Makefile.in. Thanks. Since cygmagic is executable, I don't think we actually

Re: [PATCH 1/1] cygwin: use CREATE_DEFAULT_ERROR_MODE in spawn

2020-12-08 Thread Jon Turney
On 07/12/2020 09:43, Corinna Vinschen wrote: On Dec 4 10:35, Jeremy Drake via Cygwin-patches wrote: On Fri, 4 Dec 2020, Corinna Vinschen via Cygwin-patches wrote: I'm not happy about a new CYGWIN option. Wouldn't it make sense, perhaps, to switch to CREATE_DEFAULT_ERROR_MODE for all

Re: [PATCH] Cygwin: Allow to set SO_PEERCRED zero (v2)

2020-12-08 Thread Corinna Vinschen via Cygwin-patches
On Dec 8 10:47, Corinna Vinschen via Cygwin-patches wrote: > Hi Mark, > > On Dec 7 19:25, Mark Geisert wrote: > > Hi Corinna, > > > > Corinna Vinschen via Cygwin-patches wrote: > > > On Dec 7 16:30, Corinna Vinschen via Cygwin-patches wrote: > > > > On Dec 7 02:29, Mark Geisert wrote: > > >

Re: [PATCH] Cygwin: Allow to set SO_PEERCRED zero (v2)

2020-12-08 Thread Corinna Vinschen via Cygwin-patches
Hi Mark, On Dec 7 19:25, Mark Geisert wrote: > Hi Corinna, > > Corinna Vinschen via Cygwin-patches wrote: > > On Dec 7 16:30, Corinna Vinschen via Cygwin-patches wrote: > > > On Dec 7 02:29, Mark Geisert wrote: > > > > The existing code errors as EINVAL any attempt to set a value for > > > >