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

Support compressed comps.xml created with createrepo_c >= 1.0 #5455

Merged
merged 2 commits into from Oct 23, 2023

Conversation

mattiaverga
Copy link
Contributor

@mattiaverga mattiaverga commented Aug 13, 2023

createrepo_c >= 1.0 will also compress comps.xml if a compression method is set.
This PR will add support to the sanity_check_repodata method for reading compressed file, with support for all the different compression formats that can be set on createrepo_c.

@mattiaverga mattiaverga force-pushed the fix-comps-file branch 4 times, most recently from 2c6f1d9 to 56389b6 Compare September 27, 2023 13:01
@mattiaverga mattiaverga force-pushed the fix-comps-file branch 2 times, most recently from 7cae5bc to d7c11f0 Compare October 6, 2023 13:12
@mattiaverga mattiaverga marked this pull request as ready for review October 6, 2023 13:12
@mattiaverga
Copy link
Contributor Author

Tests will need to wait until librepo 0.1.20 is pushed to stable F39 and F38

@mattiaverga mattiaverga changed the title Fix compose.xml content for failing tests Support compressed comps.xml created with createrepo_c >= 1.0 Oct 6, 2023
@mattiaverga mattiaverga added this to the 8.0 milestone Oct 14, 2023
Signed-off-by: Mattia Verga <mattia.verga@tiscali.it>
Signed-off-by: Mattia Verga <mattia.verga@tiscali.it>
@mattiaverga
Copy link
Contributor Author

F38 and Rawhide tests now are fine, so the problem should be fixed. STill waiting for F39 libcomps update to hit stable, but I'm going to merge this PR

@mattiaverga mattiaverga merged commit 65a3d67 into fedora-infra:develop Oct 23, 2023
26 of 29 checks passed
@mattiaverga mattiaverga deleted the fix-comps-file branch October 23, 2023 07:46
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.

None yet

2 participants