bpo-35727: Use exit code 0 on sys.exit() in multiprocessing.Process.

Author: chrahunt

Lib/multiprocessing/process.py

@@ -315,12 +315,12 @@ def _bootstrap(self, parent_sentinel=None):
             finally:
                 util._exit_function()
         except SystemExit as e:
-            if not e.args:
-                exitcode = 1
-            elif isinstance(e.args[0], int):
-                exitcode = e.args[0]
+            if e.code is None:
+                exitcode = 0
+            elif isinstance(e.code, int):
+                exitcode = e.code
             else:
-                sys.stderr.write(str(e.args[0]) + '\n')
+                sys.stderr.write(str(e.code) + '\n')
                 exitcode = 1
         except:
             exitcode = 1

Lib/test/_test_multiprocessing.py

@@ -855,6 +855,13 @@ def test_sys_exit(self):
             join_process(p)
             self.assertEqual(p.exitcode, reason)
 
+        for args in ((None,), (),):
+            p = self.Process(target=sys.exit, args=args)
+            p.daemon = True
+            p.start()
+            join_process(p)
+            self.assertEqual(p.exitcode, 0)
+
 #
 #
 #

Misc/ACKS

@@ -735,6 +735,7 @@ Lawrence Hudson
 Michael Hudson
 Jim Hugunin
 Greg Humphreys
+Chris Hunt
 Eric Huss
 Nehal Hussain
 Taihyun Hwang

Misc/NEWS.d/next/Library/2019-01-12-20-39-34.bpo-35727.FWrbHn.rst

@@ -0,0 +1 @@
+Fix sys.exit() and sys.exit(None) exit code propagation when used in multiprocessing.Process.