a fix from maciej who claims this fixes issues on some systems.
passes all tests so i just apply it. --HG-- branch : trunk
This commit is contained in:
@@ -18,8 +18,8 @@ def cmdexec(cmd):
|
||||
universal_newlines=True,
|
||||
stdout=subprocess.PIPE, stderr=subprocess.PIPE)
|
||||
out, err = process.communicate()
|
||||
out = py.builtin._totext(out, sys.getdefaultencoding())
|
||||
err = py.builtin._totext(err, sys.getdefaultencoding())
|
||||
out = py.builtin._totext(out, sys.stdout.encoding)
|
||||
err = py.builtin._totext(err, sys.stderr.encoding)
|
||||
status = process.poll()
|
||||
if status:
|
||||
raise ExecutionFailed(status, status, cmd, out, err)
|
||||
|
||||
Reference in New Issue
Block a user