diff options
author | Victor Stinner <victor.stinner@gmail.com> | 2017-06-08 18:34:30 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-06-08 18:34:30 +0200 |
commit | e5bdad2201de45c203037e59491e4fea56def56d (patch) | |
tree | 7a24d825a76eea6def464ecba34bc4324c8c9d28 /Lib/subprocess.py | |
parent | fe681397dc66e2bc7b5baf02cb4f22a1005c6400 (diff) | |
download | cpython-git-e5bdad2201de45c203037e59491e4fea56def56d.tar.gz |
bpo-30418: Popen.communicate() always ignore EINVAL (#2002) (#2006)
On Windows, subprocess.Popen.communicate() now also ignore EINVAL
on stdin.write() if the child process is still running but closed the
pipe.
(cherry picked from commit d52aa31378ae43e044a300edfe8285954c167216)
Diffstat (limited to 'Lib/subprocess.py')
-rw-r--r-- | Lib/subprocess.py | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/Lib/subprocess.py b/Lib/subprocess.py index c6ecc46186..4b41f5ec5c 100644 --- a/Lib/subprocess.py +++ b/Lib/subprocess.py @@ -720,10 +720,11 @@ class Popen(object): if e.errno == errno.EPIPE: # communicate() should ignore broken pipe error pass - elif (e.errno == errno.EINVAL - and self.poll() is not None): - # Issue #19612: stdin.write() fails with EINVAL - # if the process already exited before the write + elif e.errno == errno.EINVAL: + # bpo-19612, bpo-30418: On Windows, stdin.write() + # fails with EINVAL if the child process exited or + # if the child process is still running but closed + # the pipe. pass else: raise |