PTY dies when master in parent process is closed.

Package: cygwin Version: 1.7.35-1 PTY of cygwin dies when master fd in parent process is closed while a child process keeps valid master fd. To reproduce this problem: 1) Compile following C sources (Test Case 1, 2) 2) Execute them. Expected results are both: 01234567890 However, result of Test

Re: PTY dies when master in parent process is closed.

Hi Takashi, On Mar 5 21:53, Takashi Yano wrote: > Package: cygwin > Version: 1.7.35-1 > > PTY of cygwin dies when master fd in parent process > is closed while a child process keeps valid master fd. > > To reproduce this problem: > 1) Compile following C sources (Test Case 1, 2) > 2) Execute th

Re: PTY dies when master in parent process is closed.

On Mar 5 14:31, Corinna Vinschen wrote: > Hi Takashi, > > On Mar 5 21:53, Takashi Yano wrote: > > Package: cygwin > > Version: 1.7.35-1 > > > > PTY of cygwin dies when master fd in parent process > > is closed while a child process keeps valid master fd. > > > > To reproduce this problem: > >

Re: PTY dies when master in parent process is closed.

On Thu, 5 Mar 2015 14:58:39 +0100 Corinna Vinschen wrote: > I applied a patch. Please have a look. I have tested the latest CVS version, and found a new problem. With new CVS version, slave side can not detect closure of master. Please use following Test Case 3. Test Case 3 is not terminated

Re: PTY dies when master in parent process is closed.

On Fri, 6 Mar 2015 20:07:10 +0900 Takashi Yano wrote: > On Thu, 5 Mar 2015 14:58:39 +0100 > Corinna Vinschen wrote: > > > I applied a patch. Please have a look. > > I have tested the latest CVS version, and found > a new problem. > > With new CVS version, slave side can not detect > closure

Re: PTY dies when master in parent process is closed.

Hi Takashi, On Mar 8 16:37, Takashi Yano wrote: > On Fri, 6 Mar 2015 20:07:10 +0900 > Takashi Yano wrote: > > > On Thu, 5 Mar 2015 14:58:39 +0100 > > Corinna Vinschen wrote: > > > > > I applied a patch. Please have a look. > > > > I have tested the latest CVS version, and found > > a new pr

Re: PTY dies when master in parent process is closed.

Hi Takashi, On Mar 8 16:37, Takashi Yano wrote: > On Fri, 6 Mar 2015 20:07:10 +0900 > Takashi Yano wrote: > > > On Thu, 5 Mar 2015 14:58:39 +0100 > > Corinna Vinschen wrote: > > > > > I applied a patch. Please have a look. > > > > I have tested the latest CVS version, and found > > a new pr

Re: PTY dies when master in parent process is closed.

On Mar 11 17:48, Corinna Vinschen wrote: > Hi Takashi, > > On Mar 8 16:37, Takashi Yano wrote: > > On Fri, 6 Mar 2015 20:07:10 +0900 > > Takashi Yano wrote: > > > > > On Thu, 5 Mar 2015 14:58:39 +0100 > > > Corinna Vinschen wrote: > > > > > > > I applied a patch. Please have a look. > > > >

Re: PTY dies when master in parent process is closed.

On Wed, 11 Mar 2015 17:48:29 +0100 Corinna Vinschen wrote: > I'm inclined to apply this patch. I think using PeekNamedPipe this way > is pretty nice, albeit it's not atomic. It might be a good idea to add > an acquire/release_output_mutex bracket, even if that doesn't help for > cases in which

Re: PTY dies when master in parent process is closed.

On Mar 12 20:03, Takashi Yano wrote: > On Wed, 11 Mar 2015 17:48:29 +0100 > Corinna Vinschen wrote: > > > I'm inclined to apply this patch. I think using PeekNamedPipe this way > > is pretty nice, albeit it's not atomic. It might be a good idea to add > > an acquire/release_output_mutex bracket

Re: PTY dies when master in parent process is closed.

Thank you, Corinna. On Thu, 12 Mar 2015 15:54:23 +0100 Corinna Vinschen wrote: > Patch applied with an additional preceeding comment so we know why > PeekNamedPipe is called here. I have confirmed that the problems have been fixed in latest git version. -- Takashi Yano -- Problem reports:

Re: PTY dies when master in parent process is closed.

On Mar 15 12:06, Takashi Yano wrote: > Thank you, Corinna. > > On Thu, 12 Mar 2015 15:54:23 +0100 > Corinna Vinschen wrote: > > > Patch applied with an additional preceeding comment so we know why > > PeekNamedPipe is called here. > > I have confirmed that the problems have been fixed > in late