On 6/8/2015 2:09 PM, Corinna Vinschen wrote:
Hi Takashi,
You convinced me. I applied your patch. I'll generate a developer
snapshot later today. It would be nice if it could get a thorough
testing.
Thanks,
Corinna
I am testing both 32bit and 64 bit version from yesterday
and the long
On Jun 8 15:50, Marco Atzeri wrote:
On 6/8/2015 2:09 PM, Corinna Vinschen wrote:
Hi Takashi,
You convinced me. I applied your patch. I'll generate a developer
snapshot later today. It would be nice if it could get a thorough
testing.
Thanks,
Corinna
I am testing both 32bit
Hi Takashi,
On Jun 7 09:03, Takashi Yano wrote:
Hi Corinna,
On Thu, 28 May 2015 16:16:12 +0200 Corinna Vinschen wrote:
I applied this patch. I'm not comfortable with removing the SIGHUP
handling from slave::read in favor of just setting errno to EIO.
EIO seems wrong here. Not being
Hi Corinna,
On Thu, 28 May 2015 16:16:12 +0200 Corinna Vinschen wrote:
I applied this patch. I'm not comfortable with removing the SIGHUP
handling from slave::read in favor of just setting errno to EIO.
EIO seems wrong here. Not being able to access the pipe anymore
should be equivalent to
On May 27 17:17, Corinna Vinschen wrote:
On May 27 14:23, Corinna Vinschen wrote:
Hi Takashi,
On May 21 20:53, Takashi Yano wrote:
[...]
+ if (output_handle_local)
+{
+ OBJECT_BASIC_INFORMATION obi;
+ NTSTATUS status;
+ ULONG hdl_cnt = 0;
+
+
Hi Corinna,
On Wed, 27 May 2015 14:23:12 +0200
Corinna Vinschen wrote:
Isn't that racy? Consider two processes doing that at the same time.
Both calls to NtQueryObject could come up with hdl_cnt == 2 and the
problem persists.
Yes, indeed. I didn't consider enough.
Wouldn't it be safer to
On Thu, 28 May 2015 13:47:28 +0200
Corinna Vinschen wrote:
I created another version of your patch which avoids duplicating the
tested handle and makes the test-and-close-handle operation atomic:
Does that look ok? It fixes the reported problem for me.
Oops! I have just sent another patch.
Hi Corinna,
On Thu, 28 May 2015 21:09:32 +0900
Takashi Yano wrote:
On Thu, 28 May 2015 13:47:28 +0200
Corinna Vinschen wrote:
I created another version of your patch which avoids duplicating the
tested handle and makes the test-and-close-handle operation atomic:
Does that look ok?
Hi Takashi,
On May 28 21:44, Takashi Yano wrote:
Hi Corinna,
On Thu, 28 May 2015 21:09:32 +0900
Takashi Yano wrote:
On Thu, 28 May 2015 13:47:28 +0200
Corinna Vinschen wrote:
I created another version of your patch which avoids duplicating the
tested handle and makes the
Hi Takashi,
On May 21 20:53, Takashi Yano wrote:
[...]
I was looking into this problem, and found the PeekNamedPipe()
call is blocked in fhandler_pty_master::close() when the problem
occurs.
I had not noticed that,
On May 27 14:23, Corinna Vinschen wrote:
Hi Takashi,
On May 21 20:53, Takashi Yano wrote:
[...]
+ if (output_handle_local)
+{
+ OBJECT_BASIC_INFORMATION obi;
+ NTSTATUS status;
+ ULONG hdl_cnt = 0;
+
+ status = NtQueryObject (output_handle_local,
Ken Brown said:
Takashi Yano wrote:
I have made a patch attached to resolve this problem. Please have
a look.
Thanks! That fixes it for me. Michael (OP), are you able to build cygwin1.dll
and test it?
Ken
Unfortunately, I don't have the ability to build a new cygwin.dll on my
Hi Ken,
On Tue, 19 May 2015 14:16:35 -0400
Ken Brown wrote:
OK, I got the bisection to work:
32fd5b0fe1548cff79b9868e4e259853c59a44db is the first bad commit
commit 32fd5b0fe1548cff79b9868e4e259853c59a44db
Author: Takashi Yano xxx
Date: Thu Mar 12 15:48:10 2015 +0100
Let pty
Hi Takashi,
On 5/21/2015 7:53 AM, Takashi Yano wrote:
I was looking into this problem, and found the PeekNamedPipe()
call is blocked in fhandler_pty_master::close() when the problem
occurs.
I had not noticed that,
https://msdn.microsoft.com/en-us/library/windows/desktop/aa365779(v=vs.85).aspx
On 5/18/2015 1:15 PM, Michael Mauger wrote:
Since the upgrade to cygwin-2 I've been having consistent problems using emacs.
My emacs config has not changed and the most recent update to cygwin-2 has
helped. I'm using the emacs-w32 executable and have numerous remote cygwin ssh
sessions
On 5/19/2015 1:14 PM, Ken Brown wrote:
On 5/18/2015 1:15 PM, Michael Mauger wrote:
Since the upgrade to cygwin-2 I've been having consistent problems
using emacs. My emacs config has not changed and the most recent
update to cygwin-2 has helped. I'm using the emacs-w32 executable and
have
16 matches
Mail list logo