Martin Panter added the comment:

Maybe you don’t want to touch the implementation of the “older high-level API” 
for fear of subtly breaking something, but for clarification, and perhaps 
documentation, would the old functions now be equivalent to this?

def call(***):
    # Verify PIPE not in (stdout, stderr) if needed
    return run(***).returncode
def check_call(***):
    # Verify PIPE not in (stdout, stderr) if needed
    run(***, check=True)
def check_output(***):
    # Verify stderr != PIPE if needed
    return run(***, check=True, stdout=PIPE)

If they are largely equivalent, perhaps simplify the documentation of them in 
terms of run(), and move them closer to the run() documentation.

Is it worth making the CalledProcessError exception a subclass of 
CompletedProcess? They seem to be basically storing the same information.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue23342>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to