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

Replace the old getopt with argparse in pdb CLI #109164

Closed
gaogaotiantian opened this issue Sep 8, 2023 · 0 comments
Closed

Replace the old getopt with argparse in pdb CLI #109164

gaogaotiantian opened this issue Sep 8, 2023 · 0 comments
Labels
type-feature A feature request or enhancement

Comments

@gaogaotiantian
Copy link
Member

gaogaotiantian commented Sep 8, 2023 •
edited by bedevere-bot

Feature or enhancement

Proposal:

pdb is using getopt for argument parsing. getopt is pretty old and not well maintained. Some issue in #108791 could be avoided with the modern argparse module. We should replace getopt with argparse when it's easy to do so. This might ease the load in the future when more arguments are added to the CLI.

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Links to previous discussion of this feature:

No response

Linked PRs
@gaogaotiantian gaogaotiantian added the type-feature A feature request or enhancement label Sep 8, 2023
@bedevere-bot bedevere-bot mentioned this issue Sep 8, 2023
Merged
vstinner added a commit that referenced this issue Sep 22, 2023
@gaogaotiantian @AA-Turner @vstinner
gh-109164: Replace getopt with argparse in pdb (#109165)
73ccfa2 
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
Co-authored-by: Victor Stinner <vstinner@python.org>
csm10495 pushed a commit to csm10495/cpython that referenced this issue Sep 28, 2023
@gaogaotiantian @AA-Turner
@vstinner @csm10495
pythongh-109164: Replace getopt with argparse in pdb (python#109165)
7ae8f6b 
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
Co-authored-by: Victor Stinner <vstinner@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type-feature A feature request or enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vstinner @gaogaotiantian