diff options
author | Victor Stinner <victor.stinner@haypocalc.com> | 2010-05-25 22:40:38 (GMT) |
---|---|---|
committer | Victor Stinner <victor.stinner@haypocalc.com> | 2010-05-25 22:40:38 (GMT) |
commit | c3e40e0454de9c6f3e46f8a6e3409b5975835af9 (patch) | |
tree | 389885f1de22df4c98e62285c73aa9911e520958 | |
parent | 7620f66397fbbd49ed4894a80de3d56356e6273e (diff) | |
download | cpython-c3e40e0454de9c6f3e46f8a6e3409b5975835af9.zip cpython-c3e40e0454de9c6f3e46f8a6e3409b5975835af9.tar.gz cpython-c3e40e0454de9c6f3e46f8a6e3409b5975835af9.tar.bz2 |
Merged revisions 81537 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/trunk
........
r81537 | victor.stinner | 2010-05-26 00:30:32 +0200 (mer., 26 mai 2010) | 3 lines
Issue #3798: Write sys.exit() message to sys.stderr to use stderr encoding and
error handler, instead of writing to the C stderr file in utf-8
........
-rw-r--r-- | Lib/test/test_sys.py | 20 | ||||
-rw-r--r-- | Misc/NEWS | 3 | ||||
-rw-r--r-- | Python/pythonrun.c | 8 |
3 files changed, 30 insertions, 1 deletions
diff --git a/Lib/test/test_sys.py b/Lib/test/test_sys.py index 5c18006..980606d 100644 --- a/Lib/test/test_sys.py +++ b/Lib/test/test_sys.py @@ -173,6 +173,26 @@ class SysModuleTest(unittest.TestCase): "raise SystemExit(47)"]) self.assertEqual(rc, 47) + def check_exit_message(code, expected, env=None): + process = subprocess.Popen([sys.executable, "-c", code], + stderr=subprocess.PIPE, env=env) + stdout, stderr = process.communicate() + self.assertEqual(process.returncode, 1) + self.assertTrue(stderr.startswith(expected), + "%s doesn't start with %s" % (repr(stderr), repr(expected))) + + # test that stderr buffer if flushed before the exit message is written + # into stderr + check_exit_message( + r'import sys; sys.stderr.write("unflushed,"); sys.exit("message")', + b"unflushed,message") + + # test that the unicode message is encoded to the stderr encoding + env = os.environ.copy() + env['PYTHONIOENCODING'] = 'latin-1' + check_exit_message( + r'import sys; sys.exit(u"h\xe9")', + b"h\xe9", env=env) def test_getdefaultencoding(self): if test.test_support.have_unicode: @@ -12,6 +12,9 @@ What's New in Python 2.6.6 alpha 1? Core and Builtins ----------------- +- Issue #3798: Write sys.exit() message to sys.stderr to use stderr encoding + and error handler, instead of writing to the C stderr file in utf-8 + - Issue #7902: When using explicit relative import syntax, don't try implicit relative import semantics. diff --git a/Python/pythonrun.c b/Python/pythonrun.c index 0ab7165..1a7e01e 100644 --- a/Python/pythonrun.c +++ b/Python/pythonrun.c @@ -1111,7 +1111,13 @@ handle_system_exit(void) if (PyInt_Check(value)) exitcode = (int)PyInt_AsLong(value); else { - PyObject_Print(value, stderr, Py_PRINT_RAW); + PyObject *sys_stderr = PySys_GetObject("stderr"); + if (sys_stderr != NULL && sys_stderr != Py_None) { + PyFile_WriteObject(value, sys_stderr, Py_PRINT_RAW); + } else { + PyObject_Print(value, stderr, Py_PRINT_RAW); + fflush(stderr); + } PySys_WriteStderr("\n"); exitcode = 1; } |