| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
misleading error message stating that args[0] did not exist when
either the cwd or executable keyword arguments specified a path that
did not exist.
It now keeps track of if the child got as far as preexec and reports it if
not back to the parent via a special "noexec" error message value in
the error pipe so that the cwd can be blamed for a failed chdir
instead of the exec of the executable being blamed instead.
The executable is also always reported accurately when exec fails.
Unittests enhanced to cover these cases.
|
|
|
|
| |
It was exposed by 03c98d05b140 and dbbf3ccf72e8.
|
| |
|
|
|
|
|
|
| |
errors correctly.
Patch by Serhiy Storchaka.
|
| |
|
|
|
|
| |
Based on patch from Hervé Coatanhay.
|
| |
|
| |
|
|
|
|
| |
"oops" while rearranging the #defines.
|
|
|
|
|
| |
Many lack readdir64, use readdir. Only use readdir64 on solaris where
it is required to work around a solaris bug.
|
|
|
|
|
|
|
|
| |
suboptimal by closing all possible file descriptors rather than just
the open ones in the child process before exec().
It now closes only the open fds when it is possible to safely determine what
those are.
|
| |
|
| |
|
| |
|
|
|
|
| |
given as a low fd, it gets overwritten.
|
|
|
|
|
|
|
| |
stdout or stderr
in the _posixsubprocess C extension module it would unintentionally close the fds and raise
an error.
|
|
|
|
|
| |
file descriptors (0, 1, 2) are closed in the parent process. Initial
patch by Ross Lagerwall.
|
|
|
|
| |
ENOSYS.
|
| |
|
|
|
|
|
|
|
|
| |
Issue #7213: Change the close_fds default on Windows to better match the new
default on POSIX. True when possible (False if stdin/stdout/stderr are
supplied).
Update the documentation to reflect all of the above.
|
|
|
|
|
| |
the C code, using pipe2() when available. Adds unittests for close_fds and
cloexec behaviors.
|
|
|
|
| |
to list exactly which file descriptors should be kept open.
|
|
|
|
| |
you have to convert your bytearray filenames to bytes
|
|
|
|
|
|
| |
the current working directory.
Remove also a trailing space, and replace tabulation indentation by spaces.
|
|
|
|
|
|
|
|
| |
being defref'ed).
* Fixes another potential refleak of a reference to the gc
module in the unlikely odd case where gc module isenabled or disable calls
fail.
* Adds a unittest for the above case to verify behavior and lack of leaks.
|
|
|
|
|
| |
* Also fixes a potential extra DECREF of an arg in the error case within
_posixsubprocess.fork_exec() by not reusing the process_args variable.
|
|
POSIX systems with a C extension module. This is required in order for
the subprocess module to be made thread safe.
The pure python implementation is retained so that it can continue to be
used if for some reason the _posixsubprocess extension module is not
available.
The unittest executes tests on both code paths to guarantee compatibility.
* Moves PyLong_FromPid and PyLong_AsPid from posixmodule.c into longobject.h.
Code reviewed by jeffrey.yasskin at http://codereview.appspot.com/223077/show
|