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

[Steve Dower]

Change by Steve Dower :


--
resolution:  -> out of date
stage: needs patch -> resolved
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Mark Lawrence]

Change by Mark Lawrence :


--
nosy:  -BreamoreBoy

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Steve Dower]

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 tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Ezio Melotti]

Changes by Ezio Melotti :


--
nosy: +steve.dower

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Mark Lawrence]

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 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[eryksun]

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-popen-os-popen2-os-popen3

--
nosy: +eryksun

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Brian Curtin]

Changes by Brian Curtin :


--
nosy:  -brian.curtin

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Mark Lawrence]

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 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Tim Golden]

Changes by Tim Golden :


--
assignee:  -> tim.golden
nosy: +tim.golden

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Mark Lawrence]

Changes by Mark Lawrence :


--
versions:  -Python 2.6

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Brian Curtin]

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

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Ezio Melotti]

Changes by Ezio Melotti :


--
stage: patch review -> needs patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Ezio Melotti]

Changes by Ezio Melotti :


--
nosy: +ezio.melotti

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Florent Xicluna]

Changes by Florent Xicluna :


--
keywords: +buildbot

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Florent Xicluna]

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\\python.exe" -c "import sys; print sys.argv" 
>>> 42')[2].read()
'\'C:\\Python27\\python.exe" -c "import\' est pas reconnu en tant que commande 
interne ou externe, un programme executable ou un fichier de commandes.\n'

It may be related to the test_popen failure.
Actually, it seems that the outer double quotes are removed before executing 
the cmdstring:
  C:\Python27\python.exe" -c "import
 ^  ^

--
nosy: +flox

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Brian Curtin]

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 tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Brian Curtin]

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 posixmodule.c.

--
components: +Library (Lib) -Tests
keywords: +needs review
nosy: +brian.curtin
versions: +Python 2.7, Python 3.1, Python 3.2 -Python 3.0
Added file: http://bugs.python.org/file15885/issue1559298_trunk.diff

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Russ Gibson]

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 as thus:

"c:\Program Files\Python2.6\Python.exe" -u "c:\Documents and
Settings\Russ Gibson\cgi-bin\cgi.py"

>From the command line, the second case works, but the first doesn't. 
Neither work from os.popen in 2.6.2.

To make sure that popen works in all cases under windows, test_popen*
need to test with separate quotes around the command and argument.  That
will show the real problem with the current os.popen* implementation(s).

(yeah, I know, shut up and provide a patch...)

--
nosy: +squeegee

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Philip Jenvey]

Philip Jenvey  added the comment:

subprocess also needs this fix applied

Does the w9xopen command line below not need this?

--
nosy: +pjenvey

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Daniel Diniz]

Changes by Daniel Diniz :


--
components: +Tests
stage:  -> patch review
type:  -> behavior
versions: +Python 2.6, Python 3.0

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

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

[Joseph Armbruster]

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

_
___
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com