]> granicus.if.org Git - python/commitdiff
bpo-30983: eval frame rename in pep 0523 broke gdb's python extension (#2803)
authorBruno "Polaco" Penteado <polaco@gmail.com>
Mon, 14 Aug 2017 22:14:17 +0000 (23:14 +0100)
committerŁukasz Langa <lukasz@langa.pl>
Mon, 14 Aug 2017 22:14:17 +0000 (15:14 -0700)
pep 0523 renames PyEval_EvalFrameEx to _PyEval_EvalFrameDefault while the gdb python extension only looks for PyEval_EvalFrameEx to understand if it is dealing with a frame.

Final effect is that attaching gdb to a python3.6 process doesnt resolve python objects. Eg. py-list and py-bt dont work properly.

This patch fixes that. Tested locally on python3.6

Tools/gdb/libpython.py

index cc23b8402df9d5586837290e7691bc82a78ab49d..7c8874a7acea2c02d302a700978ddb89c6096261 100755 (executable)
@@ -1502,8 +1502,10 @@ class Frame(object):
         return False
 
     def is_evalframeex(self):
-        '''Is this a PyEval_EvalFrameEx frame?'''
-        if self._gdbframe.name() == 'PyEval_EvalFrameEx':
+        '''Is this a PyEval_EvalFrameEx or _PyEval_EvalFrameDefault (PEP 0523)
+        frame?'''
+        if self._gdbframe.name() in ('PyEval_EvalFrameEx',
+                                     '_PyEval_EvalFrameDefault'):
             '''
             I believe we also need to filter on the inline
             struct frame_id.inline_depth, only regarding frames with