[issue1559298] test_popen fails on Windows if installed to "Program Files"

Change by Steve Dower : -- resolution: -> out of date stage: needs patch -> resolved status: open -> closed ___ Python tracker ___

[issue1559298] test_popen fails on Windows if installed to "Program Files"

Change by Mark Lawrence : -- nosy: -BreamoreBoy ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue1559298] test_popen fails on Windows if installed to Program Files

Mark Lawrence added the comment: Do we leave this open or close it as there is a known work around for 2.7? -- versions: -Python 3.1, Python 3.2 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1559298

[issue1559298] test_popen fails on Windows if installed to Program Files

Changes by Ezio Melotti ezio.melo...@gmail.com: -- nosy: +steve.dower ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1559298 ___ ___

[issue1559298] test_popen fails on Windows if installed to Program Files

Steve Dower added the comment: The docs are pretty clear about using Popen instead of os.popen, and people using popen have likely worked around it already, so we're more likely to break them by changing it now. I vote to close. -- ___ Python

[issue1559298] test_popen fails on Windows if installed to Program Files

eryksun added the comment: This is fixed for subprocess.Popen in 2.7, 3.1, and 3.2; see issue 2304. In 2.7, nt.popen still has this problem. As mentioned above, it can be worked around by using subprocess.Popen as described here:

[issue1559298] test_popen fails on Windows if installed to Program Files

Mark Lawrence added the comment: Is this worth pursuing given the wording here https://docs.python.org/3/library/subprocess.html#replacing-os-popen-os-popen2-os-popen3 ? -- nosy: +BreamoreBoy ___ Python tracker rep...@bugs.python.org

[issue1559298] test_popen fails on Windows if installed to Program Files

Changes by Brian Curtin br...@python.org: -- nosy: -brian.curtin ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1559298 ___ ___ Python-bugs-list

[issue1559298] test_popen fails on Windows if installed to Program Files

Changes by Tim Golden m...@timgolden.me.uk: -- assignee: - tim.golden nosy: +tim.golden ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1559298 ___

[issue1559298] test_popen fails on Windows if installed to Program Files

Changes by Mark Lawrence breamore...@yahoo.co.uk: -- versions: -Python 2.6 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1559298 ___ ___

[issue1559298] test_popen fails on Windows if installed to Program Files

Changes by Ezio Melotti ezio.melo...@gmail.com: -- stage: patch review - needs patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1559298 ___ ___

[issue1559298] test_popen fails on Windows if installed to Program Files

Brian Curtin cur...@acm.org added the comment: Florent is correct. The patch seems to fix regular popen, but popen3 sees problems. I'll see if I can fit this in and have a look. Also of note is that the other flavors of popen are not tested...at least not in Lib/test/test_popen.py or

[issue1559298] test_popen fails on Windows if installed to Program Files

Changes by Ezio Melotti ezio.melo...@gmail.com: -- nosy: +ezio.melotti ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1559298 ___ ___

[issue1559298] test_popen fails on Windows if installed to Program Files

Florent Xicluna la...@yahoo.fr added the comment: It is on Python 2.7a2 os.popen('C:\\Python27\\python.exe -c import sys; print sys.argv 42').read() '' os.popen('C:\\Python27\\python.exe -c import sys; print sys.argv 42').read() ['-c', '42']\n os.popen3('C:\\Python27\\python.exe -c

[issue1559298] test_popen fails on Windows if installed to Program Files

Changes by Florent Xicluna la...@yahoo.fr: -- keywords: +buildbot ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1559298 ___ ___ Python-bugs-list

[issue1559298] test_popen fails on Windows if installed to Program Files

Brian Curtin cur...@acm.org added the comment: This has come up recently and Martin's approach seems to work. I updated his patch for trunk, and test_popen passes when I run it with and without a space in the path. Russ Gibson's suggestion was already applied to test_popen, so the only code

[issue1559298] test_popen fails on Windows if installed to Program Files

Brian Curtin cur...@acm.org added the comment: Here is a py3k version of the previous patch. os.popen is implemented using subprocess.Popen, so the quoting change was made there. -- Added file: http://bugs.python.org/file15886/issue1559298_py3k.diff

[issue1559298] test_popen fails on Windows if installed to Program Files

Russ Gibson ru...@rnstech.com added the comment: What is needed is separate quoting for the command and the argument. Right now, test_popen only surrounds the entire command line with quotes: c:\Program Files\Python2.6\Python.exe -u c:\Documents and Settings\Russ Gibson\cgi-bin\cgi.py It

[issue1559298] test_popen fails on Windows if installed to Program Files

Philip Jenvey pjen...@users.sourceforge.net added the comment: subprocess also needs this fix applied Does the w9xopen command line below not need this? -- nosy: +pjenvey ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1559298

[issue1559298] test_popen fails on Windows if installed to Program Files

Changes by Daniel Diniz aja...@gmail.com: -- components: +Tests stage: - patch review type: - behavior versions: +Python 2.6, Python 3.0 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue1559298

[issue1559298] test_popen fails on Windows if installed to Program Files

Joseph Armbruster added the comment: I applied the change to: Python 2.6a0 (trunk:58651M, Nov 18 2007, 08:46:54) [MSC v.1400 32 bit (Intel)] on win32 and test_popen passes appeared to pass. -- nosy: +JosephArmbruster _ Tracker [EMAIL PROTECTED]