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
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
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:
> >
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
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
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
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
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.
> > >
>
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
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
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:
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
12 matches
Mail list logo