Martin Panter added the comment:

I agree with the deprecation idea. The parameter checking logic doesn’t seem 
right though; see Reitveld. Also, I would have made the warning specify exactly 
what is deprecated, in case the stack trace doesn’t identify the function, 
which I think would always happen with check_call(). Also be less specific 
about future changes, unless there is clear consensus to make this change in 
3.6. Maybe something like:

"Passing PIPE to call() and check_call() is deprecated; use DEVNULL instead to 
discard output or provide empty input"

Since 3.5 is now in the beta phase, would adding this deprecation be allowed, 
or should it be deferred to the 3.6 branch? Also, I’m not sure what the policy 
is for Python 2. Maybe it would be acceptable as a Python 3 compatibility 
warning, triggered by the “python2 -3” option; I dunno.

----------
components:  -Windows
nosy: +vadmium
stage:  -> needs patch
title: subprocess.check_call hangs on large PIPEd data. -> Deprecate PIPE with 
subprocess.check_call() and call()

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

Reply via email to