-
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 doesn't install build-system.requires
from dependencies not using Poetry.
#7702
Comments
Hello @adamgreig, I'm able to reproduce it with the example repo you have prepared. But this minimal example have two problems.
Once I have fixed these two problems I cannot reproduce the issue any more. I believe that there is nothing to fix on poetrys side here. Can you confirm this? fin swimmer |
Not OP, but I managed to still reproduce the problem, as long as there is a Running
It does not matter whether I specify [tool.poetry.build]
generate-setup-file = false
script = "build.py" or [tool.poetry.build]
generate-setup-file = true
script = "build.py" or nothing in pyproject.toml. |
I have hit the exact same error @SolidTux outlines in his comment
The ldpc package in this case, does list cython as a build requirement See Here |
original reproduction was bogus, reproduction at #7702 (comment) has been removed, no reproduction in #7702 (comment) it's entirely unclear what this issue is even about: since the original bug report was wrong and there are no repros since then it's hard to say whether any of these are the "same" or whether folk are just hijacking this issue. suggest closing this out and people should raise new issues - with reproduction - for new problems. |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
-vvv
option) and have included the output below.Issue
Poetry doesn't install entries in
[build-system.requires]
from dependencies not using Poetry. In my case a dependency has a build system requirement onsetuptools_scm
, which must be installed forsetup.py
to run successfully.setuptools_scm
is listed in the dependency's[build-system.requires]
but isn't installed by Poetry, so the overall installation fails.Adding
setuptools_scm
to the dependency's actual dependencies list causes it to be installed first, and then the build succeeds.I ran into this issue trying to install amaranth but have reproduced the error in a minimal test case here. Run
poetry install
inpoetry_bug_2/
to see the error.It seems like the issue (thanks to @whitequark for investigating) is here:
poetry/src/poetry/inspection/info.py
Line 599 in 71b3d6e
which should possibly include the extra requirements:
I'm happy to prepare a PR with that change if you would like.
Full error output is below but the pertinent error is:
Output of `poetry install -vvv`
The text was updated successfully, but these errors were encountered: