-
Notifications
You must be signed in to change notification settings - Fork 51
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
Fix entrypoints management #63
Conversation
src/installer/utils.py
Outdated
@@ -190,6 +190,9 @@ def parse_entrypoints(text): | |||
|
|||
# Borrowed from https://github.com/python/importlib_metadata/blob/v3.4.0/importlib_metadata/__init__.py#L90 # noqa |
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.
Is this comment still up-to-date @pradyunsg? The linked target is entirely different from the code below.
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.
Yea, this is out of date. Let’s drop it.
@sdispater could you make this change in this PR?
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.
Actually, nvm. Let’s do that in a follow up.
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.
@pradyunsg I did it anyway :-)
Hi, when will this fix be released? I am planning to adopt |
Ah buggery. I forgot about that. I’ll do if over the coming weekend but if I don’t, please file a new issue. |
I am currently trying to integrate
installer
into Poetry and things are mostly good but there were issues with how the entrypoints were managed. More specifically:installer
was looking for aentry-points.txt
file while the actual file isentry_points.txt
).parse_entrypoints()
function only parses script entry points.