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

Downgrade pipenv from 2023.2.18 to 2023.2.4 #1425

Merged
merged 1 commit into from
Mar 10, 2023
Merged

Conversation

edmorley
Copy link
Member

@edmorley edmorley commented Mar 10, 2023

pipenv 2023.2.18 raised its minimum setuptools version to 67.x, which is newer than the setuptools version the buildpack pins to. This succeeds on the first build, however, for cached rebuilds then fails due to pip's version resolver not allowing overriding already-installed dependency versions when resolving version conflicts.

A new testcase has been added for pipenv caching (caching was already extensively tested when using pip, just not when using pipenv).

Resolves:

remote: -----> Installing pip 23.0.1, setuptools 63.4.3 and wheel 0.38.4
remote: ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
remote: pipenv 2023.2.18 requires setuptools>=67.0.0, but you have setuptools 63.4.3 which is incompatible.

Longer term we will update to newer setuptools ourselves (once the dust settles on the many breaking changes introduced in newer setuptools versions), and hopefully newer pipenv will also relax its minimum setuptools version slightly.

Partially reverts #1412.

GUS-W-12670935.

@edmorley edmorley added the bug label Mar 10, 2023
@edmorley edmorley self-assigned this Mar 10, 2023
pipenv 2023.2.18 raised its minimum setuptools version to 67.x, which is newer
than the setuptools version the buildpack pins to. This succeeds on the first build,
however, for cached rebuilds then fails due to pip's version resolver not allowing
overriding already-installed dependency versions when resolving version conflicts.

A new testcase has been added for pipenv caching (caching was already extensively
tested when using pip, just not when using pipenv).

Resolves:

```
remote: -----> Installing pip 23.0.1, setuptools 63.4.3 and wheel 0.38.4
remote: ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
remote: pipenv 2023.2.18 requires setuptools>=67.0.0, but you have setuptools 63.4.3 which is incompatible.
```

Longer term we will update to newer setuptools ourselves (once the dust settles on
the many breaking changes introduced in newer setuptools versions), and hopefully
newer pipenv will also relax its minimum setuptools version slightly.

GUS-W-12670935.
@edmorley edmorley marked this pull request as ready for review March 10, 2023 13:07
@edmorley edmorley requested a review from a team as a code owner March 10, 2023 13:07
@edmorley edmorley enabled auto-merge (squash) March 10, 2023 13:12
@edmorley edmorley merged commit c900bb9 into main Mar 10, 2023
@edmorley edmorley deleted the downgrade-pipenv branch March 10, 2023 13:48
@edmorley edmorley mentioned this pull request Mar 10, 2023
@edmorley
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants