<div dir="ltr">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.<br><div><br></div><div>-gregory.p.smith</div></div><br><div class="gmail_quote">On Thu Jan 22 2015 at 10:05:56 PM gregory.p.smith <<a href="mailto:python-checkins@python.org">python-checkins@python.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><a href="https://hg.python.org/cpython/rev/7b833bd1f509" target="_blank">https://hg.python.org/cpython/<u></u>rev/7b833bd1f509</a><br>
changeset:   94249:7b833bd1f509<br>
branch:      3.4<br>
parent:      94247:7f3ac2ce24ed<br>
user:        Gregory P. Smith <<a href="mailto:greg@krypto.org" target="_blank">greg@krypto.org</a>><br>
date:        Thu Jan 22 22:04:16 2015 -0800<br>
summary:<br>
  Remove the unimplemented but ignored without='-E' parameters being passed to<br>
script_helper.assert_python_<u></u>failure().  No such feature has ever existed,<br>
thus it doesn't do what the comment claims.  (It does add a 'without'<br>
variable to the environment of the child process but that was not intended)<br>
<br>
files:<br>
  Lib/test/test_cmd_line.py |  6 ++----<br>
  1 files changed, 2 insertions(+), 4 deletions(-)<br>
<br>
<br>
diff --git a/Lib/test/test_cmd_line.py b/Lib/test/test_cmd_line.py<br>
--- a/Lib/test/test_cmd_line.py<br>
+++ b/Lib/test/test_cmd_line.py<br>
@@ -426,13 +426,11 @@<br>
         self.assertIn(b'Unknown option: -z', err)<br>
         self.assertEqual(err.<u></u>splitlines().count(b'Unknown option: -z'), 1)<br>
         self.assertEqual(b'', out)<br>
-        # Add "without='-E'" to prevent _assert_python to append -E<br>
-        # to env_vars and change the output of stderr<br>
-        rc, out, err = assert_python_failure('-z', without='-E')<br>
+        rc, out, err = assert_python_failure('-z')<br>
         self.assertIn(b'Unknown option: -z', err)<br>
         self.assertEqual(err.<u></u>splitlines().count(b'Unknown option: -z'), 1)<br>
         self.assertEqual(b'', out)<br>
-        rc, out, err = assert_python_failure('-a', '-z', without='-E')<br>
+        rc, out, err = assert_python_failure('-a', '-z')<br>
         self.assertIn(b'Unknown option: -a', err)<br>
         # only the first unknown option is reported<br>
         self.assertNotIn(b'Unknown option: -z', err)<br>
<br>
--<br>
Repository URL: <a href="https://hg.python.org/cpython" target="_blank">https://hg.python.org/cpython</a><br>
______________________________<u></u>_________________<br>
Python-checkins mailing list<br>
<a href="mailto:Python-checkins@python.org" target="_blank">Python-checkins@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/python-checkins" target="_blank">https://mail.python.org/<u></u>mailman/listinfo/python-<u></u>checkins</a><br>
</blockquote></div>