Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org

2021-03-08 Thread Brian Inglis
On 2021-03-08 13:55, Corinna Vinschen via Cygwin-patches wrote: On Mar 8 15:20, Ken Brown via Cygwin-patches wrote: On 3/8/2021 2:09 PM, Achim Gratz wrote: Brian Inglis writes: It's normally a merge conflict which will not be satisfied by regular commands to restore the working files to

[PATCH] Cygwin: pty: Transfer input only if the stdin is a pty.

2021-03-08 Thread Takashi Yano via Cygwin-patches
- The commit 12325677f73a did not fix enough. With this patch, more transfer_input() calls are skipped if stdin is redirected or piped. --- winsup/cygwin/fhandler_tty.cc | 10 -- winsup/cygwin/spawn.cc| 9 +++-- 2 files changed, 15 insertions(+), 4 deletions(-) diff --git

Re: [PATCH] Cygwin: pty: Transfer input for native app only if the stdin is pcon.

2021-03-08 Thread Takashi Yano via Cygwin-patches
On Mon, 8 Mar 2021 21:52:37 +0100 Corinna Vinschen wrote: > On Mar 9 00:48, Takashi Yano via Cygwin-patches wrote: > > On Mon, 8 Mar 2021 16:32:16 +0100 > > Corinna Vinschen wrote: > > > Hi Takashi, > > > > > > On Mar 8 23:55, Takashi Yano via Cygwin-patches wrote: > > > > - Currently, transfer

Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org

2021-03-08 Thread Corinna Vinschen via Cygwin-patches
On Mar 8 15:20, Ken Brown via Cygwin-patches wrote: > On 3/8/2021 2:09 PM, Achim Gratz wrote: > > Brian Inglis writes: > > > It's normally a merge conflict which will not be satisfied by regular > > > commands to restore the working files to upstream. > > > > So you're pulling on an unclean work

Re: [PATCH 0/2] cygwin-htdocs: update MinGW references

2021-03-08 Thread Corinna Vinschen via Cygwin-patches
On Mar 8 09:17, Brian Inglis wrote: > update MinGW.org references and update package categories to remove MinGW > > Brian Inglis (2): > cygwin-htdocs/links.html: update MinGW.org reference > cygwin-htdocs/packaging-hint-files.html: update categories > > links.html| 4 ++-- >

Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org

2021-03-08 Thread ASSI
Ken Brown via Cygwin-patches writes: > If the unfinished business consists of local commits that haven't yet > been applied upstream, then I typically do the following: > > git fetch # Find out if upstream has changed since my last pull. If so... > git format-patch -n # save n local commits >

Re: [PATCH] Cygwin: pty: Transfer input for native app only if the stdin is pcon.

2021-03-08 Thread Corinna Vinschen via Cygwin-patches
On Mar 9 00:48, Takashi Yano via Cygwin-patches wrote: > On Mon, 8 Mar 2021 16:32:16 +0100 > Corinna Vinschen wrote: > > Hi Takashi, > > > > On Mar 8 23:55, Takashi Yano via Cygwin-patches wrote: > > > - Currently, transfer input is triggered even if the stdin of native > > > app is not a

Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org

2021-03-08 Thread Ken Brown via Cygwin-patches
On 3/8/2021 2:09 PM, Achim Gratz wrote: Brian Inglis writes: It's normally a merge conflict which will not be satisfied by regular commands to restore the working files to upstream. So you're pulling on an unclean work tree? That's a no-no, either keep your changes on a separate branch (that

Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org

2021-03-08 Thread Achim Gratz
Brian Inglis writes: > It's normally a merge conflict which will not be satisfied by regular > commands to restore the working files to upstream. So you're pulling on an unclean work tree? That's a no-no, either keep your changes on a separate branch (that you can rebase or merge later) or stash

Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org

2021-03-08 Thread Brian Inglis
On 2021-03-08 10:38, Achim Gratz wrote: Brian Inglis writes: Thanks - I'll try to remember to try that - on a pull conflict I normally try to checkout -- file(s), then -f, then origin/master, then plus -f, with status checks between, then commit -m merge when required, and re-pull origin/master

Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org

2021-03-08 Thread Achim Gratz
Brian Inglis writes: > Thanks - I'll try to remember to try that - on a pull conflict I > normally try to checkout -- file(s), then -f, then origin/master, then > plus -f, with status checks between, then commit -m merge when > required, and re-pull origin/master to check resynced to upstream >

Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org

2021-03-08 Thread Brian Inglis
On 2021-03-08 03:14, Corinna Vinschen via Cygwin-patches wrote: On Mar 7 13:26, Brian Inglis wrote: On 2021-03-07 12:15, Jon Turney wrote: On 07/03/2021 16:31, Brian Inglis wrote:   winsup/doc/dll.xml | 5 +++--   1 file changed, 3 insertions(+), 2 deletions(-) I don't think the link

[PATCH 1/2] cygwin-htdocs/links.html: update MinGW.org reference

2021-03-08 Thread Brian Inglis
--- links.html | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/links.html b/links.html index ab2bb6e5d7d5..705e60c023a8 100755 --- a/links.html +++ b/links.html @@ -22,10 +22,10 @@ as well. These binaries do not use the Cygwin DLL and only support the Windows APIs.

[PATCH 2/2] cygwin-htdocs/packaging-hint-files.html: update categories

2021-03-08 Thread Brian Inglis
--- packaging-hint-files.html | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packaging-hint-files.html b/packaging-hint-files.html index 025bbb4e5830..ceecd206737e 100755 --- a/packaging-hint-files.html +++ b/packaging-hint-files.html @@ -130,8 +130,8 @@ No actual moles

[PATCH 0/2] cygwin-htdocs: update MinGW references

2021-03-08 Thread Brian Inglis
update MinGW.org references and update package categories to remove MinGW Brian Inglis (2): cygwin-htdocs/links.html: update MinGW.org reference cygwin-htdocs/packaging-hint-files.html: update categories links.html| 4 ++-- packaging-hint-files.html | 4 ++-- 2 files

Re: [PATCH] Cygwin: pty: Transfer input for native app only if the stdin is pcon.

2021-03-08 Thread Takashi Yano via Cygwin-patches
On Mon, 8 Mar 2021 16:32:16 +0100 Corinna Vinschen wrote: > Hi Takashi, > > On Mar 8 23:55, Takashi Yano via Cygwin-patches wrote: > > - Currently, transfer input is triggered even if the stdin of native > > app is not a pseudo console. With this patch it is triggered only > > if the stdin

Re: [PATCH] Cygwin: pty: Transfer input for native app only if the stdin is pcon.

2021-03-08 Thread Corinna Vinschen via Cygwin-patches
On Mar 8 23:55, Takashi Yano via Cygwin-patches wrote: > - Currently, transfer input is triggered even if the stdin of native > app is not a pseudo console. With this patch it is triggered only > if the stdin is a pseudo console. > --- > winsup/cygwin/fhandler_tty.cc | 18 ++

Re: [PATCH] Cygwin: pty: Transfer input for native app only if the stdin is pcon.

2021-03-08 Thread Corinna Vinschen via Cygwin-patches
Hi Takashi, On Mar 8 23:55, Takashi Yano via Cygwin-patches wrote: > - Currently, transfer input is triggered even if the stdin of native > app is not a pseudo console. With this patch it is triggered only > if the stdin is a pseudo console. do you have more patches in the loop? I wonder

[PATCH] Cygwin: pty: Transfer input for native app only if the stdin is pcon.

2021-03-08 Thread Takashi Yano via Cygwin-patches
- Currently, transfer input is triggered even if the stdin of native app is not a pseudo console. With this patch it is triggered only if the stdin is a pseudo console. --- winsup/cygwin/fhandler_tty.cc | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git

Re: [PATCH] Cygwin: pty: Attach to stub process when non-cygwin app inherits pcon.

2021-03-08 Thread Corinna Vinschen via Cygwin-patches
On Mar 8 22:14, Takashi Yano via Cygwin-patches wrote: > - If two non-cygwin apps are started simultaneously, attaching to > pseudo console sometimes fails. This is because the second app > trys to attach to the process not started yet. This patch avoids > the issue by attaching to the stub

[PATCH] Cygwin: pty: Attach to stub process when non-cygwin app inherits pcon.

2021-03-08 Thread Takashi Yano via Cygwin-patches
- If two non-cygwin apps are started simultaneously, attaching to pseudo console sometimes fails. This is because the second app trys to attach to the process not started yet. This patch avoids the issue by attaching to the stub process rather than the other non-cygwin app. ---

Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org

2021-03-08 Thread Corinna Vinschen via Cygwin-patches
On Mar 7 13:26, Brian Inglis wrote: > On 2021-03-07 12:15, Jon Turney wrote: > > On 07/03/2021 16:31, Brian Inglis wrote: > > > --- > > >   winsup/doc/dll.xml | 5 +++-- > > >   1 file changed, 3 insertions(+), 2 deletions(-) > > > I don't think the link here actually has much value, and would be

Re: [PATCH] winsup/doc/dll.xml: update MinGW/.org to MinGW-w64/.org

2021-03-08 Thread Corinna Vinschen via Cygwin-patches
On Mar 7 19:15, Jon Turney wrote: > On 07/03/2021 16:31, Brian Inglis wrote: > > --- > > winsup/doc/dll.xml | 5 +++-- > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > I don't think the link here actually has much value, and would be inclined > to drop it, as far as I can tell it's

Re: [PATCH] Cygwin: console, pty: Stop ignoring Ctrl-C by IGNBRK.

2021-03-08 Thread Corinna Vinschen via Cygwin-patches
On Mar 7 10:41, Takashi Yano via Cygwin-patches wrote: > - Perhaps current code misunderstand meaning of the IGNBRK. As far > as I investigated, IGNBRK is concerned with break signal in serial > port but there is no evidence that it has effect to ignore Ctrl-C. > This patch stops ignoring

Re: [PATCH] Cygwin: pty: Discard input already accepted on interrupt.

2021-03-08 Thread Corinna Vinschen via Cygwin-patches
On Mar 5 18:01, Takashi Yano via Cygwin-patches wrote: > - Currently, input already accepted is not discarded on interrupt > by VINTR, VQUIT and VSUSP keys. This patch fixes the issue. > --- > winsup/cygwin/fhandler.h | 2 ++ > winsup/cygwin/fhandler_termios.cc | 5 - >