[issue6445] Add check parameter to subprocess.Popen.communicate

Changes by Serhiy Storchaka : -- stage: test needed -> resolved status: pending -> closed ___ Python tracker ___ ___ Python-bugs-list m

[issue6445] Add check parameter to subprocess.Popen.communicate

Martin Panter added the comment: According to PEP 375, 3.1 was out on 27 June 2009, just before this report was opened (9 July). So perhaps OG7 wasn’t aware of check_output() at the time. But it sounds like they did want CalledProcessError to be raised. I suggest to close this, unless there is

[issue6445] Add check parameter to subprocess.Popen.communicate

Martin Panter added the comment: In the mean time, a subprocess.run() API has been added which can call communicate() and raise CalledProcessError in one step (depending on arguments). See Issue 23342. This might satisfy some use cases, but maybe adding the check functionality to the low-level

[issue6445] Add check parameter to subprocess.Popen.communicate

akira added the comment: subprocess.check_output() could be used in "communicate() + check process exit status" one-liners. It returns child process output (stdout) and raises an exception if the returncode is not zero. It is available since Python 2.7 (3.1) If you don't want to raise an error f

[issue6445] Add check parameter to subprocess.Popen.communicate

Changes by Mark Lawrence : -- nosy: -BreamoreBoy ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue6445] Add check parameter to subprocess.Popen.communicate

Mark Lawrence added the comment: @OG7 this is unlikely to move unless you provide a patch that changes the code and the unit test. -- nosy: +BreamoreBoy stage: -> unit test needed type: -> feature request versions: +Python 3.2 ___ Python tracker

[issue6445] Add check parameter to subprocess.Popen.communicate

New submission from OG7 : communicate is often used in one-liners, but becomes a four-liner if you want to check the process exit status. Adding a check parameter would make it more convenient to get things right and write non-buggy code. The CalledProcessError requires a cmd argument, which mea