[Python-checkins] bpo-35931: Gracefully handle any exception in pdb debug command (GH-12103)
Miss Islington (bot)
webhook-mailer at python.org
Mon Mar 11 23:29:08 EDT 2019
https://github.com/python/cpython/commit/3e936431e23b424b1e4665e8165c245924f0ab02
commit: 3e936431e23b424b1e4665e8165c245924f0ab02
branch: master
author: Daniel Hahler <github at thequod.de>
committer: Miss Islington (bot) <31488909+miss-islington at users.noreply.github.com>
date: 2019-03-11T20:29:04-07:00
summary:
bpo-35931: Gracefully handle any exception in pdb debug command (GH-12103)
This is relevant for `debug doesnotexist()`, which would crash with a
NameError otherwise.
files:
A Misc/NEWS.d/next/Library/2019-03-11-22-06-36.bpo-35931.Qp_Tbe.rst
M Lib/pdb.py
M Lib/test/test_pdb.py
diff --git a/Lib/pdb.py b/Lib/pdb.py
index 1443f9f85c7e..bf3219af3985 100755
--- a/Lib/pdb.py
+++ b/Lib/pdb.py
@@ -1093,16 +1093,14 @@ def do_debug(self, arg):
sys.settrace(None)
globals = self.curframe.f_globals
locals = self.curframe_locals
- try:
- code = compile(arg, "<string>", "exec")
- except SyntaxError:
- exc_info = sys.exc_info()[:2]
- self.error(traceback.format_exception_only(*exc_info)[-1].strip())
- return
p = Pdb(self.completekey, self.stdin, self.stdout)
p.prompt = "(%s) " % self.prompt.strip()
self.message("ENTERING RECURSIVE DEBUGGER")
- sys.call_tracing(p.run, (code, globals, locals))
+ try:
+ sys.call_tracing(p.run, (arg, globals, locals))
+ except Exception:
+ exc_info = sys.exc_info()[:2]
+ self.error(traceback.format_exception_only(*exc_info)[-1].strip())
self.message("LEAVING RECURSIVE DEBUGGER")
sys.settrace(self.trace_dispatch)
self.lastcmd = p.lastcmd
diff --git a/Lib/test/test_pdb.py b/Lib/test/test_pdb.py
index 21f6b7079bb5..7e03df02946b 100644
--- a/Lib/test/test_pdb.py
+++ b/Lib/test/test_pdb.py
@@ -1486,12 +1486,26 @@ def test_relative_imports_on_plain_module(self):
stdout, _ = self._run_pdb(['-m', self.module_name + '.runme'], commands)
self.assertTrue(any("VAR from module" in l for l in stdout.splitlines()), stdout)
- def test_syntaxerror_in_command(self):
- commands = "print(\ndebug print("
- stdout, _ = self.run_pdb_script("", commands)
+ def test_errors_in_command(self):
+ commands = "\n".join([
+ 'print(',
+ 'debug print(',
+ 'debug doesnotexist',
+ 'c',
+ ])
+ stdout, _ = self.run_pdb_script('', commands + '\n')
+
self.assertEqual(stdout.splitlines()[1:], [
'(Pdb) *** SyntaxError: unexpected EOF while parsing',
- '(Pdb) *** SyntaxError: unexpected EOF while parsing',
+
+ '(Pdb) ENTERING RECURSIVE DEBUGGER',
+ '*** SyntaxError: unexpected EOF while parsing',
+ 'LEAVING RECURSIVE DEBUGGER',
+
+ '(Pdb) ENTERING RECURSIVE DEBUGGER',
+ '> <string>(1)<module>()',
+ "((Pdb)) *** NameError: name 'doesnotexist' is not defined",
+ 'LEAVING RECURSIVE DEBUGGER',
'(Pdb) ',
])
diff --git a/Misc/NEWS.d/next/Library/2019-03-11-22-06-36.bpo-35931.Qp_Tbe.rst b/Misc/NEWS.d/next/Library/2019-03-11-22-06-36.bpo-35931.Qp_Tbe.rst
new file mode 100644
index 000000000000..68c57e2e69df
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2019-03-11-22-06-36.bpo-35931.Qp_Tbe.rst
@@ -0,0 +1 @@
+The :mod:`pdb` ``debug`` command now gracefully handles all exceptions.
More information about the Python-checkins
mailing list