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

Remove gzip option, always set this to true #67

Closed
mauritsvanrees opened this issue Sep 9, 2022 · 0 comments · Fixed by #80
Closed

Remove gzip option, always set this to true #67

mauritsvanrees opened this issue Sep 9, 2022 · 0 comments · Fixed by #80
Assignees

Comments

@mauritsvanrees
Copy link
Member

From the readme:

Use repozo's zipping functionality. true by default. Set it to false and repozo will not gzip its files. Note that gzipped databases are called *.fsz, not *.fs.gz. Changed in 0.8: the default used to be false, but it so totally makes sense to gzip your backups that we changed the default.

Now that I write this, I remember that there is a gzip option in ZODB as well, so you store zipped versions of transactions, or something like that. This should mean that zipping the Data.fs backup would make very little difference. So perhaps we should keep the option.
If there is a lot of code that can be dropped, it could still be nice.

@mauritsvanrees mauritsvanrees self-assigned this Sep 9, 2022
mauritsvanrees added a commit that referenced this issue Feb 1, 2023
From now on, we always call repozo with the gzip option, you can no longer turn this off.
Fixes #67
mauritsvanrees added a commit that referenced this issue Feb 1, 2023
From now on, we always call repozo with the gzip option, you can no longer turn this off.
Fixes #67
mauritsvanrees added a commit that referenced this issue Feb 1, 2023
From now on, we always call repozo with the gzip option, you can no longer turn this off.
Fixes #67
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 a pull request may close this issue.

1 participant