On 28 January 2015 at 10:02, Ethan Furman <ethan@stoneleaf.us> wrote:
It seems like there's some level of interest in this; shall I work on a patch?
That would be great!
First pass at a patch attached. I also ended up adding stderr attributes to CalledProcessError and TimeoutExceeded, so information is not thrown away. Things I wasn't sure about: 1. Should stdout/stderr be captured by default? I opted not to, for consistency with Popen and with shells. 2. I gave run() a check_returncode parameter, but it feels quite a long name for a parameter. Is 'check' clear enough to use as the parameter name? 3. Popen has an 'args' attribute, while CalledProcessError and TimeoutExpired have 'cmd'. CompletedProcess sits between those cases, so which name should it use? For now, it's args. Thomas