Skip to content
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

PR: Handle internal '_pdbcmd' #513

Closed
wants to merge 1 commit into from
Closed

PR: Handle internal '_pdbcmd' #513

wants to merge 1 commit into from

Conversation

impact27
Copy link
Contributor

@impact27 impact27 commented Oct 4, 2024 •
edited
Loading

Pdb added internal commands that starts with "_pdbcmd"

fixes spyder-ide/spyder#22500

@impact27 impact27 changed the title PR: fix name '_pdbcmd_print_frame_status' is not defined PR: Handle internal '_pdbcmd' Oct 4, 2024
@impact27 impact27 mentioned this pull request Oct 4, 2024
Closed
@ccordoba12 ccordoba12 mentioned this pull request Feb 5, 2025
Merged
@ccordoba12
Copy link
Member

@impact27, I think the problem requires a more generic solution because more Pdb commands (besides _pdbcmd_* ones) could be added to Pdb's cmdqueue in the future (see python/cpython#120928 for how that command was introduced). So, we need to process them as such instead of handling this particular case differently.

That's why I opened PR #530, which I verified fixes the problem too.

@ccordoba12 ccordoba12 closed this Feb 5, 2025
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.

name '_pdbcmd_print_frame_status' is not defined message when debugging
2 participants
@impact27 @ccordoba12