[PATCH] ssh: use shlex to parse command line
pmezard at gmail.com
Mon Sep 10 17:01:51 CDT 2007
Alexis S. L. Carvalho a écrit :
> Thus spake Patrick Mézard:
>> 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) ?
> One alternative that wouldn't require shlex would be to just add some
> util.quotecommand function that adds the extra quotes on windows. From
> the user POV, he'd use the same syntax from the [hooks] section (i.e.
> system-specific quoting instead of shlex quoting). Somthing like this:
Pushed your version in crew-stable and crew as 5a65d870871d.
More information about the Mercurial-devel