diff options
author | Gregory P. Smith <greg@krypto.org> | 2012-11-11 08:08:45 (GMT) |
---|---|---|
committer | Gregory P. Smith <greg@krypto.org> | 2012-11-11 08:08:45 (GMT) |
commit | d4b645ef423667f179f136bee711b8ca85bd28d8 (patch) | |
tree | 6ec01926161be10e961087c1e445534f8e6a94f1 /Lib/subprocess.py | |
parent | 8d07c264e4bcaaf1fe3b9fc92ea5730efe13eaa2 (diff) | |
parent | 6893732c353ed7cbd2492164aa2b7c64a9f1c840 (diff) | |
download | cpython-d4b645ef423667f179f136bee711b8ca85bd28d8.zip cpython-d4b645ef423667f179f136bee711b8ca85bd28d8.tar.gz cpython-d4b645ef423667f179f136bee711b8ca85bd28d8.tar.bz2 |
Remove the subprocess "bad exception data" warning (formerly a print!)
all together and just include the repr of the data in the exception
itself instead of the useless string "Unknown".
This code path is unlikely to even be possible to take given the
nature of the pipe it gets subprocess data from.
Diffstat (limited to 'Lib/subprocess.py')
-rw-r--r-- | Lib/subprocess.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/Lib/subprocess.py b/Lib/subprocess.py index d52f9a4..0fff36f 100644 --- a/Lib/subprocess.py +++ b/Lib/subprocess.py @@ -1412,11 +1412,10 @@ class Popen(object): exception_name, hex_errno, err_msg = ( errpipe_data.split(b':', 2)) except ValueError: - warnings.warn(RuntimeWarning( - 'Bad exception data: %r' % errpipe_data)) exception_name = b'SubprocessError' hex_errno = b'0' - err_msg = b'Unknown' + err_msg = (b'Bad exception data from child: ' + + repr(errpipe_data)) child_exception_type = getattr( builtins, exception_name.decode('ascii'), SubprocessError) |