Re: [PATCH] Cygwin: termios: Set ECHOE, ECHOK, ECHOCTL and ECHOKE by default.

2020-05-19 Thread Brian Inglis
On 2020-05-17 18:50, Takashi Yano wrote: > On Mon, 18 May 2020 01:21:07 +0200, Kacper Michajlow wrote: >> On Sun, 17 May 2020 at 04:53, Takashi Yano wrote: >>> - Backspace key does not work correctly in linux session opend by >>> ssh from cygwin console if the shell is bash. This is due to lack

Re: [PATCH] Cygwin: pty: Make system_printf() work after closing pty slave.

2020-05-19 Thread Ken Brown via Cygwin-patches
Hi Takashi, On 5/19/2020 7:35 AM, Takashi Yano via Cygwin-patches wrote: - Current pty cannot show system_printf() output after closing pty slave. This patch fixes the issue. Sorry to be returning the favor so soon, but this patch causes 'make check' in the texinfo source tree to hang. I

Re: [PATCH 00/21] FIFO: Support multiple readers

2020-05-19 Thread Takashi Yano via Cygwin-patches
On Tue, 19 May 2020 09:37:17 -0400 Ken Brown via Cygwin-patches wrote: > On 5/19/2020 8:51 AM, Ken Brown via Cygwin-patches wrote: > > On 5/19/2020 2:15 AM, Takashi Yano via Cygwin-patches wrote: > >> On Tue, 19 May 2020 10:26:09 +0900 > >> Takashi Yano via Cygwin-patches wrote: > >>> Hi Ken, > >

Re: [PATCH 00/21] FIFO: Support multiple readers

2020-05-19 Thread Ken Brown via Cygwin-patches
On 5/19/2020 8:51 AM, Ken Brown via Cygwin-patches wrote: On 5/19/2020 2:15 AM, Takashi Yano via Cygwin-patches wrote: On Tue, 19 May 2020 10:26:09 +0900 Takashi Yano via Cygwin-patches wrote: Hi Ken, On Mon, 18 May 2020 13:42:19 -0400 Ken Brown via Cygwin-patches wrote: Hi Takashi, On 5/1

Re: [PATCH] Cygwin: pty: Make system_printf() work after closing pty slave.

2020-05-19 Thread Corinna Vinschen
On May 19 20:35, Takashi Yano via Cygwin-patches wrote: > - Current pty cannot show system_printf() output after closing pty > slave. This patch fixes the issue. > --- > winsup/cygwin/fhandler_tty.cc | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/winsup/cygwin/fhandler_tty.cc b/w

Re: [PATCH v2] Cygwin: pty: Call FreeConsole() only if attached to current pty.

2020-05-19 Thread Corinna Vinschen
On May 19 19:55, Takashi Yano via Cygwin-patches wrote: > - After commit 071b8e0cbd4be33449c12bb0d58f514ed8ef893c, the problem > reported in https://cygwin.com/pipermail/cygwin/2020-May/244873.html > occurs. This is due to freeing console device accidentally rather > than pseudo console. This

Re: [PATCH 00/21] FIFO: Support multiple readers

2020-05-19 Thread Ken Brown via Cygwin-patches
On 5/19/2020 2:15 AM, Takashi Yano via Cygwin-patches wrote: On Tue, 19 May 2020 10:26:09 +0900 Takashi Yano via Cygwin-patches wrote: Hi Ken, On Mon, 18 May 2020 13:42:19 -0400 Ken Brown via Cygwin-patches wrote: Hi Takashi, On 5/18/2020 12:03 PM, Ken Brown via Cygwin-patches wrote: On 5/

Re: [PATCH 3/3] Cygwin: tzcode resync v2: details

2020-05-19 Thread Corinna Vinschen
Hi Mark, On May 18 22:02, Mark Geisert wrote: > Add tz_posixrules.h with data generated from most recent Cygwin tzdata > package. Establish localtime.cc as primarily a wrapper around a patched > copy of localtime.c. See README for more information. The idea is nice, but there are a few problems

[PATCH] Cygwin: pty: Make system_printf() work after closing pty slave.

2020-05-19 Thread Takashi Yano via Cygwin-patches
- Current pty cannot show system_printf() output after closing pty slave. This patch fixes the issue. --- winsup/cygwin/fhandler_tty.cc | 5 + 1 file changed, 5 insertions(+) diff --git a/winsup/cygwin/fhandler_tty.cc b/winsup/cygwin/fhandler_tty.cc index 5a1bcd3ce..02b78cd2c 100644 --- a/w

[PATCH v2] Cygwin: pty: Call FreeConsole() only if attached to current pty.

2020-05-19 Thread Takashi Yano via Cygwin-patches
- After commit 071b8e0cbd4be33449c12bb0d58f514ed8ef893c, the problem reported in https://cygwin.com/pipermail/cygwin/2020-May/244873.html occurs. This is due to freeing console device accidentally rather than pseudo console. This patch makes sure to call FreeConsole() only if the process is

Re: Re [Cygwin PATCH */9] tzcode resync -- for discussion only

2020-05-19 Thread Corinna Vinschen
On May 19 11:48, Corinna Vinschen wrote: > On May 13 15:40, Mark Geisert wrote: > > I'm not absolutely sure yet but I think this patch set isn't complete. > > What's been posted is OK for discussion (on cygwin-developers?) but would > > need to be augmented if you're going to apply as-is. > > > >

Re: [PATCH] Cygwin: termios: Set ECHOE, ECHOK, ECHOCTL and ECHOKE by default.

2020-05-19 Thread Corinna Vinschen
On May 17 11:34, Takashi Yano via Cygwin-patches wrote: > - Backspace key does not work correctly in linux session opend by > ssh from cygwin console if the shell is bash. This is due to lack > of these flags. > > Addresses: https://cygwin.com/pipermail/cygwin/2020-May/244837.html. > --- >

Re: [PATCH] Cygwin: pty: Call FreeConsole() only if attached to current pty.

2020-05-19 Thread Corinna Vinschen
On May 17 11:34, Takashi Yano via Cygwin-patches wrote: > - After commit 071b8e0cbd4be33449c12bb0d58f514ed8ef893c, the problem > reported in https://cygwin.com/pipermail/cygwin/2020-May/244873.html > occurs. This is due to freeing console device accidentally rather > than pseudo console. This

Re: [PATCH] libc: Replace i386/sys/fenv.h symlink with an #include shim

2020-05-19 Thread Corinna Vinschen
On May 16 14:43, David Macek via Cygwin-patches wrote: > Same reasoning as fbaa0967. This patch should go to the newlib mailing list, not here. Please add a bit more context than just the SHA-1 in the commit message and ideally plewase CC Joel Sherrill, the author of the original patch. Thanks,

Re: Re [Cygwin PATCH */9] tzcode resync -- for discussion only

2020-05-19 Thread Corinna Vinschen
On May 13 15:40, Mark Geisert wrote: > I'm not absolutely sure yet but I think this patch set isn't complete. > What's been posted is OK for discussion (on cygwin-developers?) but would > need to be augmented if you're going to apply as-is. > > I had a git commit/revert mishap and these are recove

Re: [PATCH v2] cygwin: doc: Add keywords for ACE order issues

2020-05-19 Thread Corinna Vinschen
On May 13 17:34, David Macek via Cygwin-patches wrote: > Windows Explorer shows a warning with Cygwin-created DACLs, but putting > the text of the warning into Google doesn't lead to the relevant Cygwin > docs. Let's copy the warning text into the docs in the hopes of helping > confused users. Mo