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

Proof of concept: Keep only python38-createrepo_c #407

Draft
wants to merge 3 commits into
base: rpm/3.16
Choose a base branch
from

Conversation

hroncok
Copy link

@hroncok hroncok commented Mar 2, 2022

No description provided.

@theforeman-bot
Copy link
Member

Can one of the admins verify this patch?

@ekohl
Copy link
Member

ekohl commented Mar 2, 2022

Context for others: https://lists.centos.org/pipermail/centos-devel/2022-March/120250.html

I do think we'll need to keep building the libs for EL7 so it'll need to be a bit more complicated, but let's see what the build process shows
ok to test

@ekohl
Copy link
Member

ekohl commented Mar 2, 2022

Our CI requires a change in release and accompanying changelog entry. If you add those, our build system should run and we can see the results.

@ekohl
Copy link
Member

ekohl commented Mar 2, 2022

ok to test

@ekohl
Copy link
Member

ekohl commented Mar 2, 2022

@evgeni I'd like your thoughts on this.

@evgeni
Copy link
Member

evgeni commented Mar 2, 2022

We need the libs part on EL7 tho, as that comes with an ancient createrepo_c.
And we might get ahead of RHEL at some point (again) where we need a createrepo_c 0.18, which RHEL doesn't have yet.

But in general, yes, this is how we should do it ;)

@evgeni
Copy link
Member

evgeni commented Apr 4, 2022

Probably obsolete, now that pulp_rpm needs a newer createrepo than available in CentOS 8 anyways:

https://github.com/pulp/pulp_rpm/blob/10a67777685c6657263e30a6ca5550ba7cedcc48/requirements.txt#L1

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

4 participants