#16306: report only the first unknown option and add more tests. Patch by Serhiy Storchaka.

ezio-melotti · ezio-melotti · commit f624ff4b6eb2 · 2012-11-23T18:46:11.000+02:00
diff --git a/Lib/test/test_cmd_line.py b/Lib/test/test_cmd_line.py
@@ -116,14 +116,24 @@ def test_del___main__(self):
             print >>script, "del sys.modules['__main__']"
         assert_python_ok(filename)
 
-
     def test_unknown_options(self):
-        # Add "without='-E'" to prevent _assert_python append env_vars -E
-        # which changes the output of stderr
+        rc, out, err = assert_python_failure('-E', '-z')
+        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')
-        self.assertIn(b'Unknown option', err)
+        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')
+        self.assertIn(b'Unknown option: -a', err)
+        # only the first unknown option is reported
+        self.assertNotIn(b'Unknown option: -z', err)
+        self.assertEqual(err.splitlines().count(b'Unknown option: -a'), 1)
+        self.assertEqual(b'', out)
+
 
 def test_main():
     test.test_support.run_unittest(CmdLineTest)
diff --git a/Modules/main.c b/Modules/main.c
@@ -264,6 +264,7 @@ Py_Main(int argc, char **argv)
 
     /* Hash randomization needed early for all string operations
        (including -W and -X options). */
+    _PyOS_opterr = 0;  /* prevent printing the error in 1st pass */
     while ((c = _PyOS_GetOpt(argc, argv, PROGRAM_OPTS)) != EOF) {
         if (c == 'm' || c == 'c') {
             /* -c / -m is the last option: following arguments are
diff --git a/Python/getopt.c b/Python/getopt.c
@@ -41,7 +41,7 @@ static char *opt_ptr = "";
 
 void _PyOS_ResetGetOpt(void)
 {
-    _PyOS_opterr = 0;   /* prevent printing the error in 2nd loop in main.c */
+    _PyOS_opterr = 1;
     _PyOS_optind = 1;
     _PyOS_optarg = NULL;
     opt_ptr = "";

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ static char *opt_ptr = "";`
`41`	`41`
`42`	`42`	`void _PyOS_ResetGetOpt(void)`
`43`	`43`	`{`
`44`		`- _PyOS_opterr = 0; /* prevent printing the error in 2nd loop in main.c */`
	`44`	`+ _PyOS_opterr = 1;`
`45`	`45`	`_PyOS_optind = 1;`
`46`	`46`	`_PyOS_optarg = NULL;`
`47`	`47`	`opt_ptr = "";`