Skip to content

Conversation

@Harry-Lees
Copy link
Contributor

@Harry-LeesHarry-Lees commented Mar 21, 2025

Closes: #131524
Related To: #131178

This PR updates the platform CLI to use argparse which adds --help flags and a usage section which was previously unavailable.

Copy link
Member

@sobolevnsobolevn left a comment

Choose a reason for hiding this comment

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

And don't forget about a NEWS entry :)

@sobolevnsobolevn requested a review from hugovkMarch 21, 2025 17:43
Copy link
Member

@hugovkhugovk left a comment

Choose a reason for hiding this comment

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

We now have help which shows the arguments:

./python.exe -m platform -husage: python.exe -m platform [-h] [--terse] [--nonaliased] [{nonaliased,terse} ...]positional arguments:{nonaliased,terse}options: -h, --help show this help message and exit --terse --nonaliased

But doesn't say what they do. Please can you add short descriptions?

Harry-Leesand others added 4 commits March 21, 2025 22:41
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@Harry-Lees
Copy link
ContributorAuthor

But doesn't say what they do. Please can you add short descriptions?

The help message now gives a short description of each flag

usage: python.exe -m platform [-h] [--terse] [--nonaliased] [{nonaliased,terse} ...] positional arguments:{nonaliased,terse} options: -h, --help show this help message and exit --terse return only the absolute minimum information needed to identify the platform --nonaliased disable system/ OS name aliasing. If aliasing is enabled, some platforms will report system names which differ from their common names, e.g. SunOS will be reported as Solaris 

@Harry-LeesHarry-Lees requested a review from hugovkMarch 21, 2025 23:11
Harry-Leesand others added 5 commits March 23, 2025 12:42
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
@donbarbos
Copy link
Contributor

Maybe it's worth adding a Command-line Usage section to the platform documentation?

@hugovk
Copy link
Member

Maybe it's worth adding a Command-line Usage section to the platform documentation?

This can be a followup PR.

Copy link
Member

@hugovkhugovk left a comment

Choose a reason for hiding this comment

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

Thanks!

@hugovk
Copy link
Member

@donbarbos GitHub says you requested changes, is there anything outstanding on this PR?

Copy link
Contributor

@donbarbosdonbarbos left a comment

Choose a reason for hiding this comment

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

sorry, I thought I was only leaving comments

@hugovkhugovk enabled auto-merge (squash) May 3, 2025 08:55
@hugovkhugovk merged commit 77c391a into python:mainMay 3, 2025
43 checks passed
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com> Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve platform CLI

5 participants

@Harry-Lees@donbarbos@hugovk@sobolevn@picnixz