Skip to content

Draft: Working towards pdb debugging support. #499

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Draft: Working towards pdb debugging support. #499

wants to merge 14 commits into from

Conversation

andrewleech
Copy link
Contributor

@andrewleech andrewleech commented Jun 16, 2022 •
edited
Loading

Details TBD

Requires: micropython/micropython#8767

@andrewleech andrewleech mentioned this pull request Jun 16, 2022
Open
@andrewleech
Copy link
Contributor Author

andrewleech commented Jun 16, 2022 •
edited
Loading

BDB Tested with bdb-example-1.zip / https://gist.github.com/Averroes/a208c5883c601992de08
Output doesn't quite match cpython at this stage, but seems to mostly work.

@andrewleech
Copy link
Contributor Author

andrewleech commented Jun 21, 2022 •
edited
Loading 

anl@STEP: ~/micropython/ports/unix $ ./micropython-dev -m pdb ../../tests/net_inet/test_tls_sites.py
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(1)<module>()
-> try:
(Pdb) b 48
Breakpoint 1 at /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py:48
(Pdb) c
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(48)main()
-> for site in SITES:
(Pdb) s
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(48)main()
-> for site in SITES:
(Pdb) unt 50
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(50)main()
-> if isinstance(site, dict):
(Pdb) s
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(54)main()
-> try:
(Pdb) s
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(55)main()
-> test_one(site, opts)
(Pdb) s
--Call--
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(14)test_one()
-> def test_one(site, opts):
(Pdb) s
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(15)test_one()
-> ai = _socket.getaddrinfo(site, 443)
(Pdb) s
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(16)test_one()
-> addr = ai[0][-1]
(Pdb) s
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(18)test_one()
-> s = _socket.socket()
(Pdb) s
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(20)test_one()
-> try:
(Pdb) s
> /home/anl/micropython/ports/unix/../../tests/net_inet/test_tls_sites.py(21)test_one()
-> s.connect(addr)
(Pdb) p addr
*** NameError("name 'addr' isn't defined",)
(Pdb)

@andrewleech
Copy link
Contributor Author

  • As expected, printing a local variable fails.
  • breakpoints seem to be stopped on the previous line... breaking on a function name main didn't work (presumably because it tried to break on previous line number`
  • pretty easy to crash
(Pdb) exit
FATAL: uncaught NLR 0x7f68a6961fa0

@andrewleech andrewleech force-pushed the pdb branch 5 times, most recently from 949e922 to b7fceca Compare June 22, 2022 06:18
@andrewleech andrewleech mentioned this pull request Nov 8, 2023
Open
@jonnor jonnor mentioned this pull request Aug 2, 2024
Open
pi-anl and others added 14 commits June 11, 2025 09:16
@pi-anl
stdlib/os: Update method of extending builtin os.
204900b
@pi-anl
stdlib/os: Provide namedtuple response for os.stat().
63c2c03
@pi-anl
bdb: Add bdb (debugging) module from cpython 3.10.
2bb641c 
Not yet updated for micropython.
Source: https://raw.githubusercontent.com/python/cpython/3.10/Lib/bdb.py
@pi-anl
linecache: Add module from cpython 3.11.
1ef2457 
source: https://github.com/python/cpython/blob/3.11/Lib/linecache.py
@pi-anl
tokenize: Add minimal stub to support linecache.
8eced76
@pi-anl
bdb: Add initial micropython support.
1f036b1 
Requires micropython to be compiled with MICROPY_PY_SYS_SETTRACE.
Also requires micropython/micropython#8767
@HowManyOliversAreThere @pi-anl
code: Add module from cpython 3.11.
f6680d9 
source: https://github.com/python/cpython/blob/3.11/Lib/code.py
@HowManyOliversAreThere @pi-anl
codeop: Add module from cpython 3.11.
7f91a7d 
source: https://github.com/python/cpython/blob/3.11/Lib/codeop.py
@pi-anl
pdb: Add module from cpython 3.11.
d9f4463 
source: https://github.com/python/cpython/blob/3.11/Lib/pdb.py
@HowManyOliversAreThere @pi-anl
code: Initial micropython support.
5f0bac9
@HowManyOliversAreThere @pi-anl
codeop: Initial micropython support.
ac67b6e
@pi-anl
cmd: Add (unused) completekey arg for compatibility.
f1d13d3
@pi-anl
pdb: Initial micropython support.
1d5113d
@pi-anl
os.path: Add stub realpath() function.
b7c81d7
@andrewleech andrewleech mentioned this pull request Jun 11, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@andrewleech @pi-anl @HowManyOliversAreThere