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

Set strict_timestamps=False when zip is called for isolated environment #9920

Merged

Conversation

nicholasserra
Copy link
Contributor

Fixes #9910

Set strict_timestamps=False when using ZipFile for new isolated environment installs. Was triggering an inconsistent ValueError('ZIP does not support timestamps before 1980') on GitHub package installs.

https://docs.python.org/3/library/zipfile.html#zipfile-objects

@nicholasserra nicholasserra changed the title Nicholasserra/9910 zip ignore timestamp Set strict_timestamps=False when zip is called for isolated environment Apr 29, 2021
news/9910.bugfix.rst Outdated Show resolved Hide resolved
Co-authored-by: Tzu-ping Chung <[email protected]>
@nicholasserra
Copy link
Contributor Author

News file has been changed

@uranusjr
Copy link
Member

uranusjr commented May 1, 2021

Unfortunately this missed the 21.1.1 deadline. I think this counts as regression, so we’ll probably need a 21.1.2 (cc @sbidoul).

Copy link
Member

@sbidoul sbidoul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change LGTM.

I'm really curious to understand the root cause.

Does this alone warrant a bugfix release ?

@uranusjr
Copy link
Member

uranusjr commented May 3, 2021

Yes, not urgent, but this should detinitely be in one of the 21.1 releases eventually.

@sbidoul sbidoul added this to the 21.1.2 milestone May 3, 2021
@radarhere
Copy link

Don't mind me, continue doing your great work for the Python community - but do you have any sense of when 21.1.2 would be released? This month? June? Or just later this year?

@uranusjr
Copy link
Member

uranusjr commented May 9, 2021

Likely this month

src/pip/_internal/build_env.py Outdated Show resolved Hide resolved
@uranusjr uranusjr merged commit 9b3bbee into pypa:main May 21, 2021
@nicholasserra nicholasserra deleted the nicholasserra/9910-zip-ignore-timestamp branch May 21, 2021 23:41
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ValueError('ZIP does not support timestamps before 1980') on GitHub package install
4 participants