-
-
Notifications
You must be signed in to change notification settings - Fork 31.8k
Improve sysconfig
CLI
#132726
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.
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
Comments
cc @hugovk |
cc @FFY00 as the sysconfig maintainer |
This CLI feels more of a debugging tool? I'm not sure how useful tests will be in this case. Brief docs are probably enough. We could change the heading to "Command-line usage", like for others (if so, keep the existing |
done, and last question what do you think about |
Please don't document |
…nGH-133035) (cherry picked from commit 0f84f6b) Co-authored-by: Semyon Moroz <[email protected]>
…33035) (#133088) Co-authored-by: Semyon Moroz <[email protected]>
Proposal:
Currently, the
sysconfig
module's CLI is fairly hidden and inaccessible. I think there are two main issues with the module CLI.python -m sysconfig
output) like there is with some other modules e.g. random. (It is, however, listed under the Modules command-line interface section)--generate-posix-vars
, which I could only find from looking at the source in./Lib/sysconfig/__main__.py
.Inspired by #131524
I suggest adding a 1. full-fledged
CommandLineTest
class to the tests (currently there is onlytest_main
which only checks that the output is present), 2. update the documentation so that it is the same as other command line interaces, 3. useargparse
as a quality CLI toolP.S.:
sysconfig
CLI was added in January 2010 for Python 3.2 (edacea3)I'm happy to submit a PR if needed.
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
sysconfig
CLI section #133035sysconfig
CLI section (GH-133035) #133088The text was updated successfully, but these errors were encountered: