Re: [PATCH] tcsetpgrp fails unexpectedly

2011-06-01 Thread Tor Perkins
On Mon, May 30, 2011 at 02:59:04AM -0400, Christopher Faylor wrote: On Mon, Apr 04, 2011 at 09:45:09AM -0700, Tor Perkins wrote: 2011-03-28 Tor Perkins * fhandler_termios.cc (fhandler_termios::bg_check): Do not return EIO when a process group has no leader as this is allowed and does

Re: [PATCH] tcsetpgrp fails unexpectedly

2011-05-30 Thread Christopher Faylor
On Mon, Apr 04, 2011 at 09:45:09AM -0700, Tor Perkins wrote: 2011-03-28 Tor Perkins * fhandler_termios.cc (fhandler_termios::bg_check): Do not return EIO when a process group has no leader as this is allowed and does not imply an orphaned process group. Add a test for orphaned process

Re: [PATCH] tcsetpgrp fails unexpectedly

2011-05-10 Thread Corinna Vinschen
On May 5 16:07, Christopher Faylor wrote: On Thu, May 05, 2011 at 08:19:24PM +0200, Corinna Vinschen wrote: On May 5 19:23, Corinna Vinschen wrote: On May 5 13:10, Christopher Faylor wrote: On Mon, Apr 04, 2011 at 12:23:00PM -0700, Tor Perkins wrote: Thanks for the patch and the

Re: [PATCH] tcsetpgrp fails unexpectedly

2011-05-05 Thread Christopher Faylor
On Mon, Apr 04, 2011 at 12:23:00PM -0700, Tor Perkins wrote: Thanks for the patch and the report. I'll take a look at this in detail in the next couple of days. However, unfortunately, I think this is a large enough submission that it requires an assignment form. Thanks for looking into it!

Re: [PATCH] tcsetpgrp fails unexpectedly

2011-05-05 Thread Corinna Vinschen
On May 5 13:10, Christopher Faylor wrote: On Mon, Apr 04, 2011 at 12:23:00PM -0700, Tor Perkins wrote: Thanks for the patch and the report. I'll take a look at this in detail in the next couple of days. However, unfortunately, I think this is a large enough submission that it requires

Re: [PATCH] tcsetpgrp fails unexpectedly

2011-05-05 Thread Corinna Vinschen
On May 5 19:23, Corinna Vinschen wrote: On May 5 13:10, Christopher Faylor wrote: On Mon, Apr 04, 2011 at 12:23:00PM -0700, Tor Perkins wrote: Thanks for the patch and the report. I'll take a look at this in detail in the next couple of days. However, unfortunately, I think this is

Re: [PATCH] tcsetpgrp fails unexpectedly

2011-05-05 Thread Christopher Faylor
On Thu, May 05, 2011 at 08:19:24PM +0200, Corinna Vinschen wrote: On May 5 19:23, Corinna Vinschen wrote: On May 5 13:10, Christopher Faylor wrote: On Mon, Apr 04, 2011 at 12:23:00PM -0700, Tor Perkins wrote: Thanks for the patch and the report. I'll take a look at this in detail in

Re: [PATCH] tcsetpgrp fails unexpectedly

2011-04-04 Thread Christopher Faylor
On Mon, Apr 04, 2011 at 09:45:09AM -0700, Tor Perkins wrote: I think I've found two problems in fhandler_termios::bg_check(): * Cygwin's tcsetpgrp function will return EIO when the process group for the calling process has no leader. * This appears to be caused by a leaderless process