-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
gh-133143: Make information about the interpreter ABI more accessible #137476
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
base: main
Are you sure you want to change the base?
Conversation
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
d572a62
to
8890dc5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a big PR!
I'll ask you to simplify (and, alas, throw a lot of the scaffolding work away...)
ab093fc
to
defb2b2
Compare
Thanks for your review, @encukou! I think I have addressed all of your comments and, as you anticipated, it simplifies things a great deal. Would you mind taking another look? |
Looks good now, thanks! And, of course, it needs deciding on what should go here; but that's for the discussion thread. |
Agreed about the tests and further discussion. I just wanted to stabilize the implementation sufficiently to be sure to write meaningful tests. Thanks again! |
Tests are in place and easily extendable. Let's focus on the discussion and get this in 💪. |
This adds a new
sys.abi_info
object to make basic abi information more readily accessible.The basic idea is to have not just flags as originally envisioned in #133143, but rather a more pythonic object with the required information, following the model of the
sys.thread_info
andsys.int_info
object as suggested in the discussion by @encukou.📚 Documentation preview 📚: https://cpython-previews--137476.org.readthedocs.build/