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 steve.dower
Recipients BreamoreBoy, barry, brett.cannon, christopher.hogan, dstufft, eric.araujo, jaraco, mgiuca, ncoghlan, r.david.murray, steve.dower, tarek, zach.ware
Date 2015-08-21.22:30:48
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1440196252.96.0.18912203065.issue8987@psf.upfronthosting.co.za>
In-reply-to
Content
That would break paths like "\" or "D:\" ("D:" != "D:\" - the backslash is meaningful here), but I'm fairly sure _nt_quote_args could add '\\"' to the end if it ends in a backslash already (to escape the trailing slash):

     for i, arg in enumerate(args):
         if ' ' in arg:
-            args[i] = '"%s"' % arg
+            args[i] = '"%s%s"' % (arg, '\\' if arg.endswith('\\') else '')
     return args

For distutils, that's probably an okay fix since we're already quoting everything, and it doesn't try to make things too smart. But it would break people who are already escaping their own trailing backslash, and doesn't provide a workaround that will work both before and after this change. Especially for something like distutils, where people often have one setup.py for all Python versions, this is pretty important.
History
Date User Action Args
2015-08-21 22:30:54steve.dowersetrecipients: + steve.dower, barry, brett.cannon, jaraco, ncoghlan, tarek, eric.araujo, mgiuca, r.david.murray, BreamoreBoy, zach.ware, dstufft, christopher.hogan
2015-08-21 22:30:52steve.dowersetmessageid: <1440196252.96.0.18912203065.issue8987@psf.upfronthosting.co.za>
2015-08-21 22:30:52steve.dowerlinkissue8987 messages
2015-08-21 22:30:48steve.dowercreate