-
Notifications
You must be signed in to change notification settings - Fork 50
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 export --format requirements.txt …
export a commit hash of an installed repository instead of the initial used tag reference.
#276
Comments
you have exported (though I don't know whether that will make a difference here) but probably you can just treat it as requirements.txt all along |
@dimbleby thanks for the What you are suggesting is installing first the dependencies and not combining it with the package installation? |
if everything is working then fwiw I find it hard to call the exported I doubt that you will get much traction but imo it would be logical to call this a pip bug for failing to recognise that the tag and the sha represent the same thing. |
This is also a problem for me. My |
Yes it does - that's what |
Poetry is used to build a package that will be installed in a Docker image:
As of today, it was working well. But we've added a new package using this (as an example)
poetry add git+https://github.com/me/new-package#1.6.0
(new-package = {git = "https://github.com/me/new-package", rev = "1.6.0"}
is present in the[tool.poetry.dependencies]
of thepyproject.toml
) and when we tried to install the generatedwheel
by Poetry using an exportedrequirements.txt
, we ended up with this error:If we take a look at the
constraints.txt
file, we can see that line:new-package @ git+https://github.com/me/new-package@c3e22d63f50256f588bd1438eedcd761a1507a43 ; python_version >= "3.11" and python_version < "4.0"
, which doesn't match the initialgit+https://github.com/me/new-package@1.6.0
.Workarounds
If we replace this line by
new-package @ git+https://github.com/me/new-package@1.6.0
the problem disappear.The text was updated successfully, but these errors were encountered: