FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

bpo-36763: Remove _PyCoreConfig.program by vstinner · Pull Request #13373 · python/cpython · GitHub

/ cpython Public

bpo-36763: Remove _PyCoreConfig.program - #13373

Merged
vstinner merged 1 commit into
python:masterfrom
vstinner:config_remove_program
May 17, 2019
Merged

bpo-36763: Remove _PyCoreConfig.program#13373
vstinner merged 1 commit into
python:masterfrom
vstinner:config_remove_program

Conversation

vstinner commented May 17, 2019
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

Copy link
Copy Markdown
Member Author

@Yhg1s: Oops, while inspecting _PyCoreConfig to review their documentation, I understood that program is kind of redundant wih program_name. Python 3.6 used argv[0] directly to display command line usage / help. I introduced "program". But after a lot of refactoring (finished by this PR), I managed to use "program_name" which is the correct program that should be used to display the command line usage / help.

vstinner merged commit fed02e1 into python:master May 17, 2019
vstinner deleted the config_remove_program branch May 17, 2019 09:12
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL