[PATCH] fhandler_tty deadlock patch

Igor Pechtchanski pechtcha@cs.nyu.edu
Mon Oct 21 08:17:00 GMT 2002


On Mon, 21 Oct 2002, Steve O wrote:

> On Sun, Oct 20, 2002 at 11:15:47PM -0400, Igor Pechtchanski wrote:
> > However, there are a couple of problems with this patch.  For example,
> > this makes bash run from a command prompt (or a shortcut) treat every
> > character as a ^D.
>
> So every character closes bash?  I'm not able to reproduce this on
> WinXP, have an strace?

Sure.  I'm attaching an strace for 'bash --rcfile /dev/null'; the
character pressed was a space.  This affects most programs, btw (vi from a
command prompt, for example, and tcsh), but not sh, cat or less.  My guess
is that anything that uses readline is affected.  I'm running Win2k SP2,
if it makes any difference.

> > /bin/sh ignores Enter (or ^J, or ^M).
>
> Good find.  I've attached a diff that should fix this.  Unsure
> how to proceed since the original patch hasn't been applied.
> Do I resubmit the original patch or treat this one as it's own
> thing?

Thanks, I'll try it out and let you know if it works for me.

> > Running 'bash -c "echo BLAH && exit"' from
> > a command prompt works, however, running "bash -c 'echo BLAH && exit'
> > ...
>
> Curious behavior, but not related to the tty patch.  Happens
> with previous cygwin dll's and in rxvt -e cmd.
>
> Thanks,
> -steve

I see.  Well, anyway, I sent an strace for that one, hopefully someone can
figure out what's going on.
	Igor
-- 
				http://cs.nyu.edu/~pechtcha/
      |\      _,,,---,,_		pechtcha@cs.nyu.edu
ZZZzz /,`.-'`'    -.  ;-;;,_		igor@watson.ibm.com
     |,4-  ) )-,_. ,\ (  `'-'		Igor Pechtchanski
    '---''(_/--'  `-'\_) fL	a.k.a JaguaR-R-R-r-r-r-.-.-.  Meow!

"Water molecules expand as they grow warmer" (C) Popular Science, Oct'02, p.51
-------------- next part --------------
A non-text attachment was scrubbed...
Name: steve_o-paste-bash1.strace.bz2
Type: application/octet-stream
Size: 17776 bytes
Desc: 
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20021021/88e49297/attachment.obj>


More information about the Cygwin-patches mailing list