-
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 fails when path depedency (PEP 517 build) depends on binary #8749
Comments
repro does not exist or is private perhaps you forgot to declare a build requirement |
Probably the fix of #8630 wants repeating inside |
Sorry, didn't realize the repo was private, fixed. |
do you want to verify the proposed fix and submit a merge request? |
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. |
Summary
Poetry fails to build a path depends on a binary that a build requirement installed to its venv's
bin
/Scripts
directory.Minimal reproduction repo: https://github.com/konstin/poetry-maturin-error. In the repo, there is a main project and a path dependency using maturin.
maturin
installs its main binary into thebin
orScripts
directory, so the directory needs to be inPATH
. pip and build handle this, poetry currently doesn't:This works ...
... while this fails:
Error messages
Running with
-vvv
prints a stacktrace but it isn't more informative. I had to hackinto PEP517_META_BUILD to get the proper error message:
Other tools
Putting
maturin
in path manually works:Running
build
manually works:System information
-vvv
option) and have included the output below:-vvv
didn't help, i had to patch poetryThe text was updated successfully, but these errors were encountered: