Skip to content

bpo-32206: Update pdb usage to include new module option#5111

Merged
ncoghlan merged 1 commit intopython:masterfrom
mariocj89:update_pdb_usage
Jan 28, 2018
Merged

bpo-32206: Update pdb usage to include new module option#5111
ncoghlan merged 1 commit intopython:masterfrom
mariocj89:update_pdb_usage

Conversation

@mariocj89
Copy link
Contributor

@mariocj89 mariocj89 commented Jan 6, 2018

Updates the usage to include the new option. Sorry @ncoghlan, I totally missed it in the previous PR

If you think it is not important/not worth it I am happy to just close it.

https://bugs.python.org/issue32206

@ncoghlan
Copy link
Contributor

ncoghlan commented Jan 6, 2018

Good catch! There's a test case that checks the usage message is displayed as expected, so that will need to be updated appropriately (the purpose of that test is to check that pdb._usage is displayed as expected, rather than to check the contents of pdb._usage, so it would be reasonable to update the test case to re-use the module global and avoid the need to update it in two places).

Lib/pdb.py Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this will be clearer as two sentences: "Debug the Python program given by pyfile. Alternatively, an executable module or package to debug can be specified using the -m switch."

@ncoghlan ncoghlan merged commit fcf8b4c into python:master Jan 28, 2018
@mariocj89 mariocj89 deleted the update_pdb_usage branch January 29, 2018 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants