From 0d4396c04cba5ac2b66fdaa23c01db84b1b54227 Mon Sep 17 00:00:00 2001 From: "Miss Islington (bot)" <31488909+miss-islington@users.noreply.github.com> Date: Mon, 9 Sep 2019 08:20:36 -0700 Subject: [PATCH] bpo-38059: Using sys.exit() over exit() in inspect.py (GH-15666) Constants added by the site module like exit() "should not be used in programs" (cherry picked from commit e3c59a75279b0df4e7553d6f0031e202de434e43) Co-authored-by: Alan Yee --- Lib/inspect.py | 4 ++-- .../next/Library/2019-09-08-11-36-50.bpo-38059.8SA6co.rst | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) create mode 100644 Misc/NEWS.d/next/Library/2019-09-08-11-36-50.bpo-38059.8SA6co.rst diff --git a/Lib/inspect.py b/Lib/inspect.py index a616f2d49b7..0a57749ccdd 100644 --- a/Lib/inspect.py +++ b/Lib/inspect.py @@ -3118,7 +3118,7 @@ def _main(): type(exc).__name__, exc) print(msg, file=sys.stderr) - exit(2) + sys.exit(2) if has_attrs: parts = attrs.split(".") @@ -3128,7 +3128,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 new file mode 100644 index 00000000000..001952ae126 --- /dev/null +++ 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()