[issue33065] IDLE debugger: failure stepping through module loading

Change by Terry J. Reedy : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue33065] IDLE debugger: failure stepping through module loading

miss-islington added the comment: New changeset 799f8489d418b7f9207d333eac38214931bd7dcc by Miss Islington (bot) in branch '3.9': bpo-33065: Fix problem debugging user classes with __repr__ method (GH-24183) https://github.com/python/cpython/commit/799f8489d418b7f9207d333eac38214931bd7dcc

[issue33065] IDLE debugger: failure stepping through module loading

miss-islington added the comment: New changeset 5ded7efa6a7a232dd4a41e6e65e4dae47146514b by Miss Islington (bot) in branch '3.8': bpo-33065: Fix problem debugging user classes with __repr__ method (GH-24183) https://github.com/python/cpython/commit/5ded7efa6a7a232dd4a41e6e65e4dae47146514b

[issue33065] IDLE debugger: failure stepping through module loading

Change by miss-islington : -- pull_requests: +23013 pull_request: https://github.com/python/cpython/pull/24185 ___ Python tracker ___

[issue33065] IDLE debugger: failure stepping through module loading

Change by miss-islington : -- nosy: +miss-islington nosy_count: 5.0 -> 6.0 pull_requests: +23012 stage: commit review -> patch review pull_request: https://github.com/python/cpython/pull/24184 ___ Python tracker

[issue33065] IDLE debugger: failure stepping through module loading

Terry J. Reedy added the comment: New changeset 81f87bbf9f65702062021a78abd9b8f82c98a414 by Terry Jan Reedy in branch 'master': bpo-33065: Fix problem debugging user classes with __repr__ method (GH-24183) https://github.com/python/cpython/commit/81f87bbf9f65702062021a78abd9b8f82c98a414

[issue33065] IDLE debugger: failure stepping through module loading

Terry J. Reedy added the comment: I am reluctant to make changes without adding tests, and until I focused again on how to do so, because of your ping, I had no idea how to do so. The added test results in the same error, "AttributeError: 'BinData' object has no attribute 'length'" without

[issue33065] IDLE debugger: failure stepping through module loading

Change by Terry J. Reedy : -- keywords: +patch pull_requests: +23011 stage: test needed -> patch review pull_request: https://github.com/python/cpython/pull/24183 ___ Python tracker

[issue33065] IDLE debugger: failure stepping through module loading

Timothy Geiser added the comment: This issue is still open, 8.5 months after identifying the underlying cause. What needs done to get a fix for this merged? Manual testing, or adding test cases? Anything I can do to help? I have 3.9.1 successfully built on a Raspberry Pi that I can reproduce

[issue33065] IDLE debugger: failure stepping through module loading

Terry J. Reedy added the comment: SO duplicate. https://stackoverflow.com/questions/61310989/python-idle-importing-xlrd-error-generated-in-debug-mode-attributeerror-mo -- ___ Python tracker

[issue33065] IDLE debugger: failure stepping through module loading

Timothy Geiser added the comment: Looks good when testing both the minimal example and the pgpdump original case. I added the import at the top, and changed the original line 173 from value = repr(value) to value = reprlib.repr(value) This is based on lines 160 & 161 in reprlib (we need a

[issue33065] IDLE debugger: failure stepping through module loading

Change by ppperry : -- nosy: -ppperry ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue33065] IDLE debugger: failure stepping through module loading

Terry J. Reedy added the comment: Timothy, can you try editing idlelib.debugger_r, line 173, as suggested above and see if it solves the problem? -- ___ Python tracker ___

[issue33065] IDLE debugger: failure stepping through module loading

Terry J. Reedy added the comment: I believe I found the bug. For IDLE's original single process mode, still available with the -n startup option, debugger.py contains the entire debugger. The values displayed for global and local names are obtained with reprlib.Repr.repr. That in turn

[issue33065] IDLE debugger: failure stepping through module loading

Timothy Geiser added the comment: It looks like the IDLE debugger seems to call repr on objects to present in the Locals list, even before they've been properly initialized. If __repr__ needs to refer to variables that don't exist until __init__ is done (and I don't think it's unreasonable

[issue33065] IDLE debugger: failure stepping through module loading

Terry J. Reedy added the comment: Timothy, this is already more helpful than a simple 'me too'. Can you reduce pgdata.dump to a truly minimal file that exhibits the bug? and then upload it? Installing any package into my local copy of the master CPython repository is problemmatical.

[issue33065] IDLE debugger: failure stepping through module loading

Timothy Geiser added the comment: I wish I could be more helpful than to just pipe up with a "this bug affects me, too," note, but wanted to poke this bug report since it's been dormant for 14 months. With Python 3.8.2 I tried using the pgpdump module (version 1.5, installed from pip) on

[issue33065] IDLE debugger: failure stepping through module loading

Terry J. Reedy added the comment: A StackOverflow use got the same _ModuleLock message. https://stackoverflow.com/questions/54785596/debugger-in-python-freezes-over-own-built-modules -- ___ Python tracker

[issue33065] IDLE debugger: failure stepping through module loading

Change by Terry J. Reedy : -- versions: -Python 3.6 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue33065] IDLE debugger: failure stepping through module loading

ppperry added the comment: Line 59 isn't actually executed; the error comes from the trace event that gets fired before line 59, which is the first `line` event in the frame containing the uninitialized _ModuleLock. -- nosy: +ppperry ___ Python

[issue33065] IDLE debugger: failure stepping through module loading

Terry J. Reedy added the comment: The problem is not limited to user modules. In duplicate issue #34609 (not closed), the same traceback was obtained with unittest. I also got the same with asyncio. There will not be a problem if the module is already loaded in sys.modules, so that