-
Notifications
You must be signed in to change notification settings - Fork 203
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
rpm/update-config-files: zstd:chunked not enabled in Fedora yet #2208
rpm/update-config-files: zstd:chunked not enabled in Fedora yet #2208
Conversation
See: https://fedoraproject.org/wiki/Changes/zstd:chunked See: https://pagure.io/fesco/issue/3252 See: containers#2048 See: https://src.fedoraproject.org/rpms/containers-common/c/9f4dd38589309fbc189eac94367f520c105c3c1c?branch=f41 Signed-off-by: Timothée Ravier <tim@siosm.fr>
9f0e257
to
1ad44cb
Compare
Sure, LGTM. We threw this in an RPM patch so the latest F41 builds are fine. |
@lsm5 PTAL |
LGTM |
Yes, saw that earlier. I was thinking "let's push it upstream so that we don't forget it". |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Luap99, travier The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
sorry for the late response. LGTM |
Already added to the Fedora package. @travier has fixed this on main in containers#2208. Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
Additional followup in #2213 |
Already added to the Fedora package. Fixed in main with containers#2208. Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
Fixed in main with containers#2208 and containers#2213. Signed-off-by: Lokesh Mandvekar <lsm5@fedoraproject.org>
See: https://fedoraproject.org/wiki/Changes/zstd:chunked
See: https://pagure.io/fesco/issue/3252
See: #2048
See: https://src.fedoraproject.org/rpms/containers-common/c/9f4dd38589309fbc189eac94367f520c105c3c1c?branch=f41