Marc-Andre Lemburg <[EMAIL PROTECTED]> added the comment:

Using subprocess is only possible conditionally, ie. if available.
Please note that the module is intended to be usable with multiple
Python versions and must at least support Python 2.1.

We could consider an updated patch for 2.7 and 3.1, but not for 2.6/3.0,
since it needs more testing.

I think for 2.6/3.0, adding the extra quotes is enough to get things
working.

Please open a new patch ticket for your patch and update it to work with
popen if subprocess is not available. The patch should also not remove
the lines:

-    if sys.platform in ('dos','win32','win16','os2'):
-        # XXX Others too ?
-        return default

from _syscmd_file().

I'll checkin the extra quotes.

_______________________________________
Python tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue3719>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to