Mike Pomraning added the comment:

If I understand correctly, the _try_wait mechanics (or 3.5's syscall behavior) 
already handle EINTR the way we way:  ignore it and try wait()ing again.

So, this patch would kill only on a timeout, and never on another error like 
Ctrl-C, a UserDefinedTimeoutException from a signal handler, etc.

That's probably the lesser of two evils, the other being a SIGKILL against an 
arbitrary child process.  Better to document that a non-timeout-parameter 
interruption to subprocess.call will separate the parent from its child, than 
to hard kill arbitrary programs when a polite SIGINT was intended.

----------

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

Reply via email to