Re: [PATCH] Cygwin: sigproc.cc: fix typo in comment describing nprocs

2020-08-28 Thread Corinna Vinschen
On Aug 27 18:40, Ken Brown via Cygwin-patches wrote: > nprocs is the number of children, not the number of deceased children. > The incorrect comment used to apply to a variable nzombies. The > latter was removed in commit 8cb359d9 in 2004, but the comment was > never updated. > --- > winsup/cygw

Re: [PATCH 0/3] CI update

2020-08-28 Thread Corinna Vinschen
On Aug 27 13:29, Brian Inglis wrote: > On 2020-08-27 02:49, Corinna Vinschen wrote: > > On Aug 26 22:04, Jon Turney wrote: > >> Since we recently had the unpleasant surprise of discovering that Cygwin > >> doesn't build on F32 when trying to make a release, this adds some CI to > >> test that. > >>

Re: [PATCH 0/3] CI update

2020-08-28 Thread Corinna Vinschen
On Aug 26 22:04, Jon Turney wrote: > Since we recently had the unpleasant surprise of discovering that Cygwin > doesn't build on F32 when trying to make a release, this adds some CI to > test that. > > Open issues: Since there don't seem to be RedHat packages for cocom, this > grabs a cocom packag

Re: [PATCH] Cygwin: select: Fix a bug on closing pi->bye event.

2020-08-28 Thread Corinna Vinschen
On Aug 27 18:46, Takashi Yano via Cygwin-patches wrote: > - Close event handle pi->bye only if it was created. > Addresses: > https://cygwin.com/pipermail/cygwin-developers/2020-August/011948.html > --- > winsup/cygwin/select.cc | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) Pu

Re: [PATCH] Cygwin: pty: Disable pseudo console if TERM is dumb or not set.

2020-08-28 Thread Corinna Vinschen
Hi Takashi, On Aug 26 21:00, Takashi Yano via Cygwin-patches wrote: > Pseudo console generates escape sequences on execution of non-cygwin > apps. If the terminal does not support escape sequence, output will > be garbled. This patch prevents garbled output in dumb terminal by > disabling pseudo

Re: [PATCH 0/3] CI update

2020-08-28 Thread Brian Inglis
On 2020-08-28 02:43, Corinna Vinschen wrote: > On Aug 27 13:29, Brian Inglis wrote: >> On 2020-08-27 02:49, Corinna Vinschen wrote: >>> On Aug 26 22:04, Jon Turney wrote: Since we recently had the unpleasant surprise of discovering that Cygwin doesn't build on F32 when trying to make a re

Re: [PATCH] Cygwin: pty: Disable pseudo console if TERM is dumb or not set.

2020-08-28 Thread Takashi Yano via Cygwin-patches
Hi Corinna, On Fri, 28 Aug 2020 15:45:03 +0200 Corinna Vinschen wrote: > Hi Takashi, > > On Aug 26 21:00, Takashi Yano via Cygwin-patches wrote: > > Pseudo console generates escape sequences on execution of non-cygwin > > apps. If the terminal does not support escape sequence, output will > > be

[PATCH v2] Cygwin: pty: Disable pseudo console if TERM is dumb or not set.

2020-08-28 Thread Takashi Yano via Cygwin-patches
- Pseudo console internally sends escape sequence CSI6n (query cursor position) on startup of non-cygwin apps. If the terminal does not support CSI6n, CreateProcess() hangs waiting for response. This patch prevents hang in dumb terminal by disabling pseudo console. --- winsup/cygwin/fhandler