[issue11377] Deprecate (remove?) platform.popen()
report at bugs.python.org
Wed Mar 2 23:42:30 CET 2011
Marc-Andre Lemburg <mal at egenix.com> added the comment:
STINNER Victor wrote:
> STINNER Victor <victor.stinner at haypocalc.com> added the comment:
> os.popen() is deprecated since Python 2.6 ("Use the subprocess module.") and it is no more documented in Python 3. The following documentation explains how to replace os.popen() by subprocess:
>> Does it prevent a shell window from opening on Windows ?
> Yes, but only if shell=True.
>> Does subprocess.Popen() use the system's PATH for finding the
> executable ?
> Yes. The subprocess module calls os.get_exec_path() which reads PATH environment variable.
One more question:
Doe subprocess work from inside Windows GUI applications ?
(I added the _popen class to work around issues with those
a long while back)
>> Since it's a documented API, we could replace it with an
>> implementation that uses subprocess.Popen(),
> platform_popen.patch patchs platform.popen() to reuse os.popen() (which uses subprocess). It adds also tests (there was no test for platform.popen)
Hmm, but if os.popen() is no longer supported in Python 3, how can
we still use it in platform ?
The os module still provides an popen() function which uses subprocess.
The documentation for the os module says:
Run child processes, returning opened pipes for communications. These functions are described in
section File Object Creation.
If you then go to the suggested section, there's no mention of os.popen().
popen() is also referenced in a few other places in the os module
The documentation should either be removed completely, or restored to
match the existing os module function popen().
>> but not remove it.
> Can't we at deprecate platform.popen in favor of subprocess, with a documentation to explain how to port code to subprocess?
Sure, just not remove it immediately.
> My patch doesn't touch platform documentation.
> os.popen() does still exist in Python 3.2, but it is no more documented. Its documentation should be fixed or the function should be removed.
Indeed. See above.
::: Try our new mxODBC.Connect Python Database Interface for free ! ::::
eGenix.com Software, Skills and Services GmbH Pastor-Loeh-Str.48
D-40764 Langenfeld, Germany. CEO Dipl.-Math. Marc-Andre Lemburg
Registered at Amtsgericht Duesseldorf: HRB 46611
Python tracker <report at bugs.python.org>
More information about the Python-bugs-list