-
Notifications
You must be signed in to change notification settings - Fork 55
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
Use python3-createrepo_c and remove some build dependencies #506
Comments
Confirm with @dralley |
We can't use the version in CentOS 8 stream, the comment is a bit out of date, we require a newer version than that. CentOS 9 stream should work, I think. But it does remove some flexibility as in the future we will need to wait for the package to be updated before using new functionality, or receiving bugfixes. In general I think that might be perfectly OK, just be aware of the tradeoff. |
@dralley Yeah, the version in CentOS 8 Stream is 0.17.7. I'd like to update the comment. What version is needed? |
0.20.0 is the minimum I believe. |
When we install the python3-createrepo_c package seems that it install the dependencies needed for the build of the createrepo_c package from pypi. With that we can remove the build dependencies and yet keep installed the createrepo_c version from pypi. What do you think @dralley? |
Not if the dependencies change between versions, which they do occasionally. Better to just use an explicit list. If we update the base image then we can just use the package, but otherwise we shouldn't mix and match. |
@decko What's the blocker on just updating the base image? EL9 isn't really bleeding edge anymore. |
I'm gonna close this in favor of #537 |
After reading the following note from our
Containerfile.core.base
manifestI went to
https://pkgs.org/download/python3-createrepo_c
website just to discover that CentOS8 Stream have the version cited in the note.So, I'm proposing a review of the packages installed into our core image for building and use Pulp.
The text was updated successfully, but these errors were encountered: