8000 gh-124703: Change back to raising bdb.BdbQuit when exiting pdb in 'inline' mode in a REPL session by ADThomas-astro · Pull Request #130395 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

gh-124703: Change back to raising bdb.BdbQuit when exiting pdb in 'inline' mode in a REPL session #130395

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.

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

Merged
merged 14 commits into from
Feb 25, 2025

Conversation

ADThomas-astro
Copy link
Contributor
@ADThomas-astro ADThomas-astro commented Feb 21, 2025

This change should mean it's again possible to call breakpoint() in a REPL session and return to the REPL session with q/quit/exit/EOF.

The test has a slight workaround/hack because I couldn't figure out how to call breakpoint() and simulate using pdb interactively within a simulated REPL session in a subprocess. After set_trace() was called, pdb would receive an EOF and exit before it could receive further commands.

If there's interest, I could also add the code from PR129768 to shorten the BdbQuit traceback by one frame.

@bedevere-app
Copy link
bedevere-app bot commented Feb 21, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@gaogaotiantian gaogaotiantian merged commit ccb4ad9 into python:main Feb 25, 2025
43 checks passed
@gaogaotiantian
Copy link
Member

Thank you for the PR!

@ADThomas-astro ADThomas-astro deleted the gh124703_fix branch February 25, 2025 04:48
@ADThomas-astro
Copy link
Contributor Author

Thanks for the merge! And thanks for being super responsive and all your hard work on pdb!

seehwan pushed a commit to seehwan/cpython that referenced this pull request Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Si 5D36 gn in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0