Skip to content
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

Prefer stdlib importlib.metdata #508

Closed
wants to merge 1 commit into from

Conversation

jonringer
Copy link

Importlib_metadata is available as part of the standard library in python >= "3.8".

Noticed it was failing in NixOS/nixpkgs#119819 (comment)

@mitya57
Copy link
Collaborator

mitya57 commented Apr 20, 2021

Please see #503 (comment).

@jonringer
Copy link
Author

hmm, interesting.

@jonringer
Copy link
Author

... What's the point of having a backport package if it exports a different API?

Anyway, I'll just a make a note in our packaging about why importlib-metadata is still needed for all stable cpython versions.

@jonringer jonringer closed this Apr 20, 2021
@jonringer jonringer deleted the fix-importlib branch April 20, 2021 18:00
@mitya57
Copy link
Collaborator

mitya57 commented Apr 20, 2021

... What's the point of having a backport package if it exports a different API?

It is backport of the API that will be present in Python 3.10.

@jonringer
Copy link
Author

Can we update https://pypi.org/project/importlib-metadata/ to also mention that the package also backports changes relevant to python_version < "3.10"

@mitya57
Copy link
Collaborator

mitya57 commented Apr 20, 2021

@jaraco ☝️ can you do it?

@jaraco
Copy link
Owner

jaraco commented Apr 20, 2021

Agreed. The readme is unclear. I filed python/importlib_metadata#306 to track.

Sign up for free to 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.

None yet

3 participants