Brian Curtin cur...@acm.org added the comment:
This change causes failures in test_httpservers on trunk r76211 on
Windows. I attached a simple patch which fixes the problem (NameError on
'p').
--
nosy: +brian.curtin
Added file: http://bugs.python.org/file15309/issue1235_fix.patch
Senthil Kumaran orsent...@gmail.com added the comment:
Thanks for the notification. Seems I over looked it. Fixed it with the
changes in 76212 (trunk) and 76213 (release26-maint). py3k did not have
this issue.
--
___
Python tracker
Senthil Kumaran orsent...@gmail.com added the comment:
drukker: I verified your patch. Looks fine for changes made to the
subprocess call.
You mentioned: I've also unified the Unix and Windows portions in
run_cgi. Both now use subprocess.
I don't see it in the patch. Was it that you assumed
Changes by Derk Drukker derk.druk...@gmail.com:
Removed file: http://bugs.python.org/file15181/fix-issue1235-py3k.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1235
___
Derk Drukker derk.druk...@gmail.com added the comment:
orsenthil: I reverted the changes that make Windows and Unix use the
same code in Patch Set 3, and briefly mentioned the reason for that in
the Rietveld app. I should have mentioned it here too.
It was because it actually makes the
Senthil Kumaran orsent...@gmail.com added the comment:
Committed revision 76208 in trunk. Shall look at the output of windows
buildbots and merge the changes to other codelines.
--
___
Python tracker rep...@bugs.python.org
Senthil Kumaran orsent...@gmail.com added the comment:
Committed revision 76209 - release26-maint
Committed revision 76210 - py3k
Committed revision 76211 - release31-maint
Thanks Derk for the patch.
--
resolution: - fixed
status: open - closed
___
Derk Drukker derk.druk...@gmail.com added the comment:
More elegant than using quotes in the cmdline string is, IMO, passing a
sequence to subprocess.Popen instead. I've put a new patch up for
review: http://codereview.appspot.com/141050
I've also unified the Unix and Windows portions in
Derk Drukker derk.druk...@gmail.com added the comment:
Salut,
Note that in py3k, CGIHTTPRequestHandler (Lib/http/server.py) already
switched to using subprocess.
AFAICT, you didn't actually change cmdline, and so the problem remains.
--
nosy: +drukker
versions: +Python 3.1, Python 3.2
Stac stac_agen...@yahoo.fr added the comment:
Oups,
I really feel sorry.
Well here is a version with the modified cmdline.
I will check how the migration was done in py3k.
Just a quick question : why the changes done in py3k was not reported to
py2.x ?
Regards,
Stac
--
Added file:
Stac stac_agen...@yahoo.fr added the comment:
Ok,
I corrected the two versions.
Now the one for py2.x is closer to the fix for py3k.
You'll find below the fixed code for both version.
Regards,
Stac
--
Added file: http://bugs.python.org/file15180/fix.tar
Derk Drukker derk.druk...@gmail.com added the comment:
The original problem reported concerned spaces in the path of the python
executable. So interp needs to be in quotes, as well (as Andy already
suggested). I've created a patch against the latest py3k, which I've
confirmed works and fixes
Changes by Stac stac_agen...@yahoo.fr:
Removed file: http://bugs.python.org/file15175/MyCGIHTTPServer.py
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1235
___
Changes by Guido van Rossum gu...@python.org:
--
nosy: -gvanrossum
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1235
___
___
Python-bugs-list
Stac stac_agen...@yahoo.fr added the comment:
Hello,
I am new to Python issues tracker. I had the error described above. I
worked on it.
I was not able to solve the issue only using subprocess. The command has
to be correctly formated (with the double quotes).
I corrected the cmdline
Christian Heimes added the comment:
The quoting issue on Windows can easily be fixed by using subprocess
instead of popen()
--
nosy: +tiran
priority: - low
versions: +Python 2.5, Python 2.6, Python 3.0 -Python 2.3
__
Tracker [EMAIL PROTECTED]
Guido van Rossum added the comment:
Please submit a patch -- this was mostly intended as demo code, I didn't
expect anyone to use it, let alone on Windows. :-)
--
nosy: +gvanrossum
__
Tracker [EMAIL PROTECTED]
http://bugs.python.org/issue1235
New submission from Andy Schumann:
On Windows a path can contain spaces (e.g. C:\Program Files\python22
\python.exe). If using popenx each component in a command line has
therefore to be double quoted as well as the whole command string.
The method run_cgi() of the class CGIHTTPRequestHandler
18 matches
Mail list logo