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 georg.brandl
Recipients asksol, benjamin.peterson, djc, georg.brandl, jnoller, tim.golden
Date 2010-12-03.20:39:10
SpamBayes Score 1.976654e-07
Marked as misclassified No
Message-id <1291408751.89.0.752893405325.issue10618@psf.upfronthosting.co.za>
In-reply-to
Content
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

(Adding 2.7 release manager to nosy.)
History
Date User Action Args
2010-12-03 20:39:11georg.brandlsetrecipients: + georg.brandl, tim.golden, benjamin.peterson, djc, jnoller, asksol
2010-12-03 20:39:11georg.brandlsetmessageid: <1291408751.89.0.752893405325.issue10618@psf.upfronthosting.co.za>
2010-12-03 20:39:10georg.brandllinkissue10618 messages
2010-12-03 20:39:10georg.brandlcreate