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 cvrebert
Recipients Neil Muller, amaury.forgeotdarc, benjamin.peterson, cvrebert, eric.araujo, ezio.melotti, georg.brandl, gvanrossum, krawyoti, lemburg, vstinner
Date 2011-05-30.10:57:25
SpamBayes Score 0.00025659686
Marked as misclassified No
Message-id <1306753049.42.0.911578509385.issue6490@psf.upfronthosting.co.za>
In-reply-to
Content
Per msg129958, attached is my stab at a patch to replace most uses of os.popen() with the subprocess module. The test suite passes on my Mac, but the patch does touch some specific-to-other-platform code, so further testing is obviously needed.
This is my first non-docs patch, please be gentle. :) [Those patches were to subprocess' docs though!]

Stuff still using os.popen() that the patch doesn't fix:
- multiprocessing
- platform.popen() [which is itself deprecated]
- subprocess.check_output()
- Lib/test/test_poll.py
- Lib/test/test_select.py
- Lib/distutils/tests/test_cygwinccompiler.py

Also, I suppose Issue 9382 should be marked as a dupe of this one?
History
Date User Action Args
2011-05-30 10:57:30cvrebertsetrecipients: + cvrebert, lemburg, gvanrossum, georg.brandl, amaury.forgeotdarc, vstinner, benjamin.peterson, ezio.melotti, Neil Muller, eric.araujo, krawyoti
2011-05-30 10:57:29cvrebertsetmessageid: <1306753049.42.0.911578509385.issue6490@psf.upfronthosting.co.za>
2011-05-30 10:57:28cvrebertlinkissue6490 messages
2011-05-30 10:57:28cvrebertcreate