bpo-38059: Using sys.exit() over exit() in inspect.py (GH-15666)

miss-islington · alanyee · web-flow · commit b7bf632d4eba · 2019-09-09T08:19:13.000-07:00
Constants added by the site module like exit() "should not be used in programs" (cherry picked from commit e3c59a7) Co-authored-by: Alan Yee <alanyee@users.noreply.github.com>
diff --git a/Lib/inspect.py b/Lib/inspect.py
@@ -3108,7 +3108,7 @@ def _main():
                                                     type(exc).__name__,
                                                     exc)
         print(msg, file=sys.stderr)
-        exit(2)
+        sys.exit(2)
 
     if has_attrs:
         parts = attrs.split(".")
@@ -3118,7 +3118,7 @@ def _main():
 
     if module.__name__ in sys.builtin_module_names:
         print("Can't get info for builtin modules.", file=sys.stderr)
-        exit(1)
+        sys.exit(1)
 
     if args.details:
         print('Target: {}'.format(target))
diff --git a/Misc/NEWS.d/next/Library/2019-09-08-11-36-50.bpo-38059.8SA6co.rst b/Misc/NEWS.d/next/Library/2019-09-08-11-36-50.bpo-38059.8SA6co.rst
@@ -0,0 +1 @@
+inspect.py now uses sys.exit() instead of exit()

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+inspect.py now uses sys.exit() instead of exit()`