Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Resolver.resolve function signature changes in pip 20.1 #59

Closed
techalchemy opened this issue Mar 5, 2020 · 0 comments · Fixed by #57
Closed

Resolver.resolve function signature changes in pip 20.1 #59

techalchemy opened this issue Mar 5, 2020 · 0 comments · Fixed by #57
Labels
bug Something isn't working

Comments

@techalchemy
Copy link
Member

Starting with pip>=20.1 calls to Resolver.resolve will move from accepting a RequirementSet instance to root_reqs: List[InstallRequirement], check_supports_wheel: bool.

@techalchemy techalchemy added the bug Something isn't working label Mar 5, 2020
techalchemy added a commit that referenced this issue Mar 5, 2020
- Add `wheel_cache` context manager helper for managing global context
  when creating wheel wheel_cache instances
- Add optional `check_supports_wheel` argument to `Resolver.resolve()`
  call when expected arguments include this one
- Inspect `InstallRequirement.ensure_build_location()` method and
  include `autodelete` kwarg if it is expected
- Update argument types passed into `resolver.resolve` which, as of
  `pip>=20.1` will expect a list of `InstallRequirement` instances
- Update `Resolver` import path to point at new location
  (`resolution.legacy.resolver`)
- Add necessary `global_tempdir_manager` contexts in tests
- Fix expected `RequirementSet.cleanup()` call after
  `Resolver.resolve()` which no longer applies due to use of transient
  `RequirementSet`
- Fixes #58
- Fixes #59
- Fixes #60
- Fixes #61
- Fixes #62

Signed-off-by: Dan Ryan <dan.ryan@canonical.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant