-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
poetry.script extras seems to be ignored, and scripts are always installed #6892
Comments
Please provide a complete pyproject.toml as per the issue template. Thanks! |
the extras are dropped here but if you use the apparently undeprecated either way, there seems to be no code that tries to avoid installing the script at I think that would happen, if it happened, in this block. Currently I don't think the probably not too hard to make this work, if you are motivated to do so. That this is so thoroughly broken and no-one has previously noticed suggests that you may be the only person in the world who might be motivated by this...! |
please keep in mind this would only work if the user is using Poetry, |
That's probably a good reason not to introduce such option. |
putting some flesh on that: https://packaging.python.org/en/latest/specifications/entry-points/ says
So perhaps an MR that cleaned up the code so as not even to try and do this would be just as welcome as one that made it work |
I will close #8841, but it's worth a look for those following this issue, as it has additional details and a repo for reproducibility. |
Closing, since it was clearly settled that extra-based entrypoints are no longer valid spec and poetry-core removed support for that with changes related to PEP 621 support. |
-vvv
option) and have included the output below.Issue
I'm trying to install a CLI script only if a given extra is supplied. Here's the config:
My understand is that
textual
will be installed on the path only if the "dev" extras has been requested.After running
poetry install
there is atextual
on the path, which is broken because of missing dependancies. I would expect that poetry would only installtextual
withpoetry install --extras dev
.The text was updated successfully, but these errors were encountered: