Re: [Python-Dev] [Python-checkins] cpython (3.4): Remove the unimplemented but ignored without='-E' parameters being passed to
I misinterpreted the side effect this code was triggering. It turns out that any kwarg would have the desired effect as a side effect (as seen in dummyvar='' being passed later in this file for the same purpose). I'm looking at refactoring various uses of -E that interfere with running the test suite in an environment where you need PYTHONHOME set. I will revert this change or otherwise do the equivalent of reinstating its behavior in a followup check-in as appropriate. -gregory.p.smith On Thu Jan 22 2015 at 10:05:56 PM gregory.p.smith < python-checkins@python.org> wrote:
https://hg.python.org/cpython/rev/7b833bd1f509 changeset: 94249:7b833bd1f509 branch: 3.4 parent: 94247:7f3ac2ce24ed user: Gregory P. Smith
date: Thu Jan 22 22:04:16 2015 -0800 summary: Remove the unimplemented but ignored without='-E' parameters being passed to script_helper.assert_python_failure(). No such feature has ever existed, thus it doesn't do what the comment claims. (It does add a 'without' variable to the environment of the child process but that was not intended) files: Lib/test/test_cmd_line.py | 6 ++---- 1 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/Lib/test/test_cmd_line.py b/Lib/test/test_cmd_line.py --- a/Lib/test/test_cmd_line.py +++ b/Lib/test/test_cmd_line.py @@ -426,13 +426,11 @@ self.assertIn(b'Unknown option: -z', err) self.assertEqual(err.splitlines().count(b'Unknown option: -z'), 1) self.assertEqual(b'', out) - # Add "without='-E'" to prevent _assert_python to append -E - # to env_vars and change the output of stderr - rc, out, err = assert_python_failure('-z', without='-E') + rc, out, err = assert_python_failure('-z') self.assertIn(b'Unknown option: -z', err) self.assertEqual(err.splitlines().count(b'Unknown option: -z'), 1) self.assertEqual(b'', out) - rc, out, err = assert_python_failure('-a', '-z', without='-E') + rc, out, err = assert_python_failure('-a', '-z') self.assertIn(b'Unknown option: -a', err) # only the first unknown option is reported self.assertNotIn(b'Unknown option: -z', err)
-- Repository URL: https://hg.python.org/cpython _______________________________________________ Python-checkins mailing list Python-checkins@python.org https://mail.python.org/mailman/listinfo/python-checkins
participants (1)
-
Gregory P. Smith