Skip to content
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

Release 1.1.6 #3905

Merged
merged 2 commits into from
Apr 14, 2021
Merged

Release 1.1.6 #3905

merged 2 commits into from
Apr 14, 2021

Conversation

sdispater
Copy link
Member

@sdispater sdispater commented Apr 9, 2021

Fixed

  • Fixed export format for path dependencies. (#3121)
  • Fixed errors caused by environment modification when executing some commands. (#3253)
  • Fixed handling of wheel files with single-digit versions. (#3338)
  • Fixed an error when handling single-digit Python markers. (poetry-core#156)
  • Fixed dependency markers not being properly copied when changing the constraint leading to resolution errors. (poetry-core#163)
  • Fixed an error where VCS dependencies were always updated. (#3947)
  • Fixed an error where the incorrect version of a package was locked when using environment markers. (#3945)

@sdispater sdispater added the kind/release Meta-issues/PRs as part of the release process label Apr 9, 2021
@sdispater sdispater requested a review from a team April 9, 2021 16:18
@abn
Copy link
Member

abn commented Apr 9, 2021

@sdispater
Copy link
Member Author

@abn It should not fail since markers are present and correct. I'll investigate but for now I will hold off the release.

@sdispater sdispater marked this pull request as draft April 9, 2021 21:37
@sdispater
Copy link
Member Author

@abn I think we are hitting a limit of the current resolver. And I don't see a simple solution to this issue.

I am surprised we did not hit this sooner but basically the issue comes from these two dependencies:

virtualenv = { version = "^20.0.26" }
pre-commit = { version = "^2.6", python = "^3.6.1" }

The resolver chooses to resolve pre-commit first: version 2.10.1 requires Python >= 3.6.1 and depends on importlib-resources (*) ; python_version < \"3.7\". That works for the resolver and it selects importlib-resources (5.1.2) which requires Python >= 3.6.

The resolver then resolves virtualenv which requires importlib-resources (>=1.0) ; python_version < \"3.7\". But the resolver in its current version assumes that importlib-resources (5.1.2) satisfies this constraint, which is only the case for Python >=3.6.1.

@sdispater
Copy link
Member Author

Well I may have a workaround it seems by prioritizing dependencies without markers first. This is not a silver bullet though and we might need to improve this part of the resolver.

@abn
Copy link
Member

abn commented Apr 14, 2021

IIUC, sounds almost like the resolver needs to keep the resolution context around from the previous result. In this case, knowing that importlib-resources (5.1.2) was resolved for >=3.6 should mean we resolve again for the difference right? This should essentially lead us to two different versions with mutually exclusive python version constraints.

@sdispater
Copy link
Member Author

@abn I tried something similar but that did not lead anywhere due to the way the resolver is structured, or it led to severe performance issues.

The workaround (#3944) should alleviate – and actually fix in most cases – the issue.

@sdispater sdispater force-pushed the release-1.1.6 branch 2 times, most recently from f32a5ca to d6af179 Compare April 14, 2021 14:11
@sdispater sdispater marked this pull request as ready for review April 14, 2021 15:00
@sdispater sdispater merged commit aa8e486 into 1.1 Apr 14, 2021
@sdispater sdispater deleted the release-1.1.6 branch April 14, 2021 17:09
Copy link

github-actions bot commented Mar 1, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/release Meta-issues/PRs as part of the release process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants