[PATCH] ssh: use shlex to parse command line
Steve Borho
steve at borho.org
Sat Sep 8 16:07:33 CDT 2007
On Sat, 2007-09-08 at 22:55 +0200, Patrick Mézard wrote:
> Steve Borho a écrit :
> > ... which looks pretty good. It seems os.popen3() is successfully
> > parsing the ssh command name, but the rest of the argument parsing
> > (probably a function of the windows shell) is broken.
>
> What about the attached patch (against crew-stable) ?
>
> Tests pass under windows and macosx but they do not exercize it very
> much and it's hard to build one not looking too kludgy. I think quoting
> all arguments should be harmless under unix but I may be wrong.
Your patch looks pretty similar to mine, so I don't understand why it
would work when mine didn't. What is your ssh command set to under
Windows? Does it start with a quoted command name?
--
Steve Borho (steve at borho.org)
http://www.borho.org/~steve/steve.asc
Key fingerprint = 2D08 E7CF B624 624C DE1F E2E4 B0C2 5292 F2C6 2C8C
More information about the Mercurial-devel
mailing list