-
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
new installer does not respect get_requires_for_build_editable #7574
Labels
Comments
onerandomusername
added
kind/bug
Something isn't working as expected
status/triage
This issue needs to be triaged
labels
Feb 28, 2023
radoering
added
area/installer
Related to the dependency installer
and removed
status/triage
This issue needs to be triaged
labels
Feb 28, 2023
LGTM, PR welcome. 😄 |
2 tasks
4 tasks
This was referenced Mar 7, 2023
4 tasks
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. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
feat/build-with-pdm-backend
branch (or the master branch if it has been merged)-vvv
option) and have included the output below.Issue
Simply put, the new installer never calls
get_requires_for_build_editable
.That's an issue with this code:
poetry/src/poetry/installation/chef.py
Lines 128 to 137 in 0e72a55
I've fixed it by changing it to the following snippet, and I'm willing to make a pr after this gets triaged 😄
This was tested by running
poetry install
with the dependency from the above pyproject.tomlThe full traceback for completeness, but I'm afraid this may not be helpful.
The text was updated successfully, but these errors were encountered: