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

Ensure timestamps of files in ZIP archives are within years 1980-2107 #7450

Draft
wants to merge 1 commit into
base: maint
Choose a base branch
from

Conversation

adswa
Copy link
Member

@adswa adswa commented Jul 3, 2023

This is done using zipfile.ZipFile's strict_timestamps option, which can be used as soon as Python 3.8 is the minimum supported Python version (i.e., it needs #7449 first, thus a draft PR) . Fixes #3753.

This is done using zipfile.ZipFile's strict_timestamps option, which can be
used as soon as Python 3.8 is the minimum supported Python version.
Fixes datalad#3753.
@adswa
Copy link
Member Author

adswa commented Jul 3, 2023

I have cancelled CI runs as their Python versions still contain Python 3.7 runs, but I lacked permissions to cancel Travis.

@yarikoptic
Copy link
Member

@adswa should this PR be updated/taken out of the draft now that 3.8 is the minimal supported?

@yarikoptic
Copy link
Member

sorry -- for some reason I thought we already merged #7449 , we haven't!

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

Successfully merging this pull request may close these issues.

old timestamp error in datalad export-to-figshare
2 participants