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

Replace remaining usage of createrepo with createrepo_c #1291

Closed
wants to merge 1 commit into from

Conversation

rohanpm
Copy link

@rohanpm rohanpm commented Mar 4, 2019

From Fedora 29 onwards, createrepo is Obsoleted by createrepo_c.
Having pulp_rpm depend on both createrepo and createrepo_c
thus blocks usage or development there. Let's move the last
remaining createrepo code over to createrepo_c to resolve this.

closes #4492
https://pulp.plan.io/issues/4492

From Fedora 29 onwards, createrepo is Obsoleted by createrepo_c.
Having pulp_rpm depend on both createrepo and createrepo_c
thus blocks usage or development there.  Let's move the last
remaining createrepo code over to createrepo_c to resolve this.

closes #4492
https://pulp.plan.io/issues/4492
@pulpbot
Copy link
Member

pulpbot commented Mar 4, 2019

Can one of the admins verify this patch?

1 similar comment
@pulpbot
Copy link
Member

pulpbot commented Mar 4, 2019

Can one of the admins verify this patch?

@goosemania
Copy link
Member

ok test

@rohanpm
Copy link
Author

rohanpm commented Mar 5, 2019

That CI job is failing because python-createrepo_c isn't installed.

How do I get it installed to the CI environment? I have a pulp-packaging PR already, https://github.com/pulp/pulp-packaging/pull/85/files

@pulpbot
Copy link
Member

pulpbot commented Mar 9, 2019

Can one of the admins verify this patch?

1 similar comment
@pulpbot
Copy link
Member

pulpbot commented Mar 9, 2019

Can one of the admins verify this patch?

@ipanova
Copy link
Member

ipanova commented Mar 12, 2019

Thank you for opening a PR!
After reaching the reporter it was determined and confirmed that the main goal of this PR is to make the dev env working. Not too long ago we decided not to support pulp on fedora 29+ due to compatibility with the latest versions of dependencies and amount of dev. resources invested into this.
So we encourage using fedora 28 for dev env.

@ipanova ipanova closed this Mar 12, 2019
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