This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author benjamin.peterson
Recipients benjamin.peterson, djc, georg.brandl, tim.golden
Date 2010-12-04.02:21:36
SpamBayes Score 1.2478981e-08
Marked as misclassified No
Message-id <AANLkTimGVB4=Py9u-ch-Zy6AHjUcyiubXxr4W-U3dfeq@mail.gmail.com>
In-reply-to <1291408751.89.0.752893405325.issue10618@psf.upfronthosting.co.za>
Content
2010/12/3 Georg Brandl <report@bugs.python.org>:
>
> Georg Brandl <georg@python.org> added the comment:
>
> In util.system(), Mercurial adds its own pair of quotes:
>
>    if os.name == 'nt':
>        cmd = '"%s"' % cmd
>
> That will result in one level of quoting too much.
>
> Now it seems unfortunate that this change was done in a minor version.
> It is definitely a bug fix, but one that many users have already worked around, probably in the same way as Mercurial.
>
> Possible ways to resolve:
>
> * make addition of quotes Python-version-specific in Mercurial
> * revert to old behavior in Python 2.7.2 (ugly)
> * add a check for quotes around the string in Python 2.7.2, and refrain from adding another set of quotes

I think we should just leave it alone. If it change the behavior again
in 2.7.2, it'll be even more consistent than before.
History
Date User Action Args
2010-12-04 02:21:37benjamin.petersonsetrecipients: + benjamin.peterson, georg.brandl, tim.golden, djc
2010-12-04 02:21:36benjamin.petersonlinkissue10618 messages
2010-12-04 02:21:36benjamin.petersoncreate