-
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 does not resolve dependencies correctly for fastdiff #3140
Comments
On previous version of
|
This commit fixed this for my repo: wemake-services/wemake-python-styleguide@22f9873 But, this was my manual hack 🙂 |
Can you please try the fix at #3147. Using pipxpipx install --suffix=@3147 'poetry @git+https://github.com/python-poetry/poetry.git@refs/pull/3147/head' Using a container (podman | docker)podman run --rm -i --entrypoint bash python:3.8 <<EOF
set -e
python -m pip install -q git+https://github.com/python-poetry/poetry.git@refs/pull/3147/head
python -m poetry new foobar
pushd foobar
sed -i /pytest/d pyproject.toml
python -m poetry add fastdiff
EOF |
It now generates:
|
Looks like it works! Thanks a lot @abn! |
If I understand correctly, the fix for this bug in #3147 was released in Poetry 1.1.3? If yes, please close this bug. |
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).Issue
Here's a lock file for this project:
Take a look at this part:
That's how
poetry
resolves deps forfastdiff
. And there's how they are defined:Source: https://github.com/syrusakbary/fastdiff/blob/master/python/setup.py
So, I end up with a missing
wasmer
package on linux. PR with reproduction: wemake-services/wemake-python-styleguide#1655The text was updated successfully, but these errors were encountered: