[issue12493] subprocess: Popen.communicate() doesn't handle EINTR in some cases

2011-07-06 Thread STINNER Victor
STINNER Victor added the comment: > Out of curiosity, how could SIGALRM be missing on a Unix system? It is only missing on Windows. -- ___ Python tracker ___ __

[issue12493] subprocess: Popen.communicate() doesn't handle EINTR in some cases

2011-07-05 Thread Charles-François Natali
Charles-François Natali added the comment: Out of curiosity, how could SIGALRM be missing on a Unix system? Maybe you mean something like @unittest.skipUnless(hasattr(errno, EINTR), "Requires errno.EINTR") -- nosy: +neologix ___

[issue12493] subprocess: Popen.communicate() doesn't handle EINTR in some cases

2011-07-05 Thread Roundup Robot
Roundup Robot added the comment: New changeset 807921ba241d by Victor Stinner in branch '3.2': Issue #12493: skip test_communicate_eintr() if signal.SIGALRM is missing http://hg.python.org/cpython/rev/807921ba241d New changeset 4928cf093a11 by Victor Stinner in branch 'default': (merge 3.2) Iss

[issue12493] subprocess: Popen.communicate() doesn't handle EINTR in some cases

2011-07-05 Thread STINNER Victor
Changes by STINNER Victor : -- resolution: -> fixed status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list U

[issue12493] subprocess: Popen.communicate() doesn't handle EINTR in some cases

2011-07-05 Thread Roundup Robot
Roundup Robot added the comment: New changeset dcfacc2d93b4 by Victor Stinner in branch '3.2': Issue #12493: subprocess: communicate() handles EINTR http://hg.python.org/cpython/rev/dcfacc2d93b4 New changeset 42e23db3ddfc by Victor Stinner in branch 'default': (merge 3.2) Issue #12493: subproce

[issue12493] subprocess: Popen.communicate() doesn't handle EINTR in some cases

2011-07-04 Thread STINNER Victor
New submission from STINNER Victor : subprocess.Popen.communicate() doesn't catch EINTR error if it is called without timeout and if there is only one PIPE (stdout or stderr). Attached patch fixes these cases. It may need a test. I found this bug while working on test_signal, especially on the