[issue4036] Support bytes for subprocess.Popen()

2010-04-20 Thread STINNER Victor 

STINNER Victor victor.stin...@haypocalc.com added the comment:

subprocess accepts byte string in program arguments in py3k, so the issue can 
be closed.

I opened other issues for the other arguments:
 - current working directory: #8393
 - environment variables: #8391
 - error message: #8467

--

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

[issue4036] Support bytes for subprocess.Popen()

2010-04-20 Thread STINNER Victor 

Changes by STINNER Victor victor.stin...@haypocalc.com:


--
resolution:  - fixed
status: open - closed

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

[issue4036] Support bytes for subprocess.Popen()

2010-03-03 Thread STINNER Victor 

STINNER Victor victor.stin...@haypocalc.com added the comment:

os.exec*() now accept bytes thanks to the PEP 383: see issue #4035.

I updated my patch: it now includes tests \o/ It works on Linux. Can someone 
test it on Windows and/or Mac OS X?

--
resolution:  - fixed
status: open - closed
Added file: http://bugs.python.org/file16423/subprocess-bytes-2.patch

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

[issue4036] Support bytes for subprocess.Popen()

2010-03-03 Thread STINNER Victor 

STINNER Victor victor.stin...@haypocalc.com added the comment:

Oops, I realized that the second test is just useless. The argument is str, not 
bytes. I wanted to test Popen(bytes, shell=True). What is the right encoding to 
convert a string to bytes for the file system?

--

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

[issue4036] Support bytes for subprocess.Popen()

2010-03-03 Thread Brian Curtin 

Brian Curtin cur...@acm.org added the comment:

The list2cmdline function checks spaces, tabs, etc in line 521 of 
subprocess.py. In your first test case, it ends up checking if a string is 
contained in a bytes object, which is a TypeError for the str not supporting 
the buffer API. 

Would it be acceptable to just add arg = str(arg) right under the for loop? 
That fixes it for me, not sure of the preferred way to go about it.

--

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

[issue4036] Support bytes for subprocess.Popen()

2010-03-03 Thread Martin v . Löwis 

Martin v. Löwis mar...@v.loewis.de added the comment:

On Windows, command lines shouldn't need to be encoded in any encoding. 
Instead, the unicode string should be passed to the system call as-is.

--

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

[issue4036] Support bytes for subprocess.Popen()

2009-05-16 Thread Daniel Diniz 

Changes by Daniel Diniz aja...@gmail.com:


--
components: +Library (Lib)
priority:  - high
type:  - feature request
versions: +Python 3.1 -Python 3.0

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

[issue4036] Support bytes for subprocess.Popen()

2008-10-30 Thread STINNER Victor 

STINNER Victor [EMAIL PROTECTED] added the comment:

@loewis: My patch only changes the behaviour of the POSIX version of 
subprocess, Windows version is unchanged.

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4036
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4036] Support bytes for subprocess.Popen()

2008-10-15 Thread Toshio Kuratomi 

Changes by Toshio Kuratomi [EMAIL PROTECTED]:


--
nosy: +a.badger

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4036
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4036] Support bytes for subprocess.Popen()

2008-10-07 Thread STINNER Victor 

Changes by STINNER Victor [EMAIL PROTECTED]:


Removed file: http://bugs.python.org/file11696/os_exec_bytes.patch

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4036
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4036] Support bytes for subprocess.Popen()

2008-10-07 Thread STINNER Victor 

STINNER Victor [EMAIL PROTECTED] added the comment:

Oops, I attached the wrong patch :-/

--
dependencies: +Support bytes for os.exec*()
Added file: http://bugs.python.org/file11742/subprocess-bytes.patch

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4036
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4036] Support bytes for subprocess.Popen()

2008-10-04 Thread Martin v. Löwis 

Martin v. Löwis [EMAIL PROTECTED] added the comment:

I think this should be deferred to 3.1. I'm not sure how much
consistency across platforms is desirable, however, using strings is
surely the better choice on Windows. I could sympathize with providing
bytes support only on POSIX, but I'm sure others will disagree.

--
nosy: +loewis

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4036
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue4036] Support bytes for subprocess.Popen()

2008-10-03 Thread STINNER Victor 

New submission from STINNER Victor [EMAIL PROTECTED]:

subprocess doesn't support bytes for the args argument.
 - On Windows, subprocess._execute_child() converts args to a string 
if it was a list
 - On UNIX, subprocess._execute_child() converts args to a list if 
it's a string

If shell=True, _execute_child() adds a prefix to the arguments.

I don't know if subprocess should accept bytes in a command line. 
Attached patch fixes POSIX version of subprocess to support bytes.

See also related issue #4035.

--
files: os_exec_bytes.patch
keywords: patch, patch
messages: 74284
nosy: haypo
severity: normal
status: open
title: Support bytes for subprocess.Popen()
versions: Python 3.0
Added file: http://bugs.python.org/file11696/os_exec_bytes.patch

___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue4036
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com