[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: https://ma

[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 track

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

Changes by Ezio Melotti : -- nosy: +steve.dower ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.

[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 ___ ___

[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: https://docs.python.org/2/library/subprocess.html#replacing-os-p

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

Changes by Brian Curtin : -- nosy: -brian.curtin ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mai

[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

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

Changes by Tim Golden : -- assignee: -> tim.golden nosy: +tim.golden ___ Python tracker ___ ___ Python-bugs-list mailing list Unsub

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

Changes by Mark Lawrence : -- versions: -Python 2.6 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://m

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

Brian Curtin 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 Lib/test/test_os.py --

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

Changes by Ezio Melotti : -- stage: patch review -> needs patch ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe

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

Changes by Ezio Melotti : -- nosy: +ezio.melotti ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

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

Changes by Florent Xicluna : -- keywords: +buildbot ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://ma

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

Florent Xicluna 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\\pyth

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

Brian Curtin 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 ___ Python tra

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

Brian Curtin 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 change is to p

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

Russ Gibson 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 needs to the above a

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

Philip Jenvey added the comment: subprocess also needs this fix applied Does the w9xopen command line below not need this? -- nosy: +pjenvey ___ Python tracker ___ __

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

Changes by Daniel Diniz : -- components: +Tests stage: -> patch review type: -> behavior versions: +Python 2.6, Python 3.0 ___ Python tracker ___

[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]>