Re: [RFA] patch for run.exe -- ATTN: ago

2006-05-21 Thread Charles Wilson
The attached patch seems to solve all the issues revealed so far. There is one open question, which may dictate a later change but the current patch should work as desired on all platforms, and is suitable for run-1.1.10 right now, IMO. Here's the open question: I can find no reliable mecha

Re: [RFA] patch for run.exe -- ATTN: ago

2006-05-21 Thread Alexander Gottwald
On Fri, 2006-05-19 at 04:42 -0400, Charles Wilson wrote: > ??? the pipe code? or the setup_invisible_console code? or both? Actually I meant the setup_invisible_console code but I'll wait until you've got further information on the handle stuff. bye ago signature.asc Description: Th

Re: [RFA] patch for run.exe -- ATTN: ago

2006-05-19 Thread Igor Peshansky
On Fri, 19 May 2006, Charles Wilson wrote: > Alexander Gottwald wrote: > > > Actually GetStdHandle seems to work with pipes. > > > > run cat foo | less cat foo | run cat | less > > but this does exit immediatly > > run cat | less > > ... > > > So the invisible console should provide stdin, stdout

Re: [RFA] patch for run.exe -- ATTN: ago

2006-05-19 Thread Charles Wilson
Alexander Gottwald wrote: Actually GetStdHandle seems to work with pipes. run cat foo | less cat foo | run cat | less but this does exit immediatly run cat | less ... So the invisible console should provide stdin, stdout and stderr where stdout and stderr will discard any written data

Re: [RFA] patch for run.exe -- ATTN: ago

2006-05-19 Thread Charles Wilson
Christopher Faylor wrote: Hopefully the run code is GPLed... Yep. Has the big ol' GPL header right at the top of run.c: /* run -- Wrapper program for console mode programs under Windows(TM) * Copyright (C) 1998 Charles S. Wilson * * This program is free software; you can redistribute it

Re: [RFA] patch for run.exe -- ATTN: ago

2006-05-18 Thread Christopher Faylor
On Thu, May 18, 2006 at 02:45:23AM -0400, Charles Wilson wrote: >As detailed here: >http://cygwin.com/ml/cygwin/2006-05/msg00471.html > >run.exe currently launches clients without any stdio handles and without >any console at all. That's over-ambitious: what run REALLY should do is >launch the c

Re: [RFA] patch for run.exe -- ATTN: ago

2006-05-18 Thread Alexander Gottwald
On Thu, 2006-05-18 at 02:45 -0400, Charles Wilson wrote: > run.exe currently launches clients without any stdio handles and without > any console at all. That's over-ambitious: what run REALLY should do is > launch the client with a *hidden* console and *working* stdio handles. > > Since run.ex