-
Notifications
You must be signed in to change notification settings - Fork 90
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
Don't set obsoletes < 5 for ELN/RHEL yet #1309
Conversation
Can you tell us what are "The necessary pieces are not yet in place for RHEL 11" so that we know when we can do the switch? Besides your patch is not syntactically correct and all builds fail ("macro expansion did not return an integer: (%2) |
Guessing from the screenshot, it's that comps groups list "yum" as a mandatory package? |
I honestly don't know what the "necessary pieces" are, but since they're not there for |
There are conflicts with yum. This should be adjusted back once things are properly addressed in Fedora. Signed-off-by: Stephen Gallagher <[email protected]>
787c958
to
c24d7d2
Compare
For more context: we want DNF5 to be available in Fedora ELN because we want to switch to it for mock/Koji build purposes. We'll move to it as a replacement for DNF4 at the same time Rawhide does. |
I'm unable to reproduce the error with a dnf5 nightly build. The ODCS compose mentioned in the screenshot does not contain dnf5 package. Koji build root for ELN contains dnf5 and it obsoletes yum, but installing "yum", or "yum dnf5" does not fail. If I add a repository with a group requiring "yum", installing that group also pass. Isn't the error a feature of Anaconda? |
The new patch looks good. After finishing rpm-build:fedora-rawhide-x86_64 test, I will merge it. |
It's entirely possible that there's something happening in |
I know that a bunch of things choke on the fact we're using Provides instead of subpackages. I've been tempted to break out the dnf, microdnf, and yum provides back into subpackages to deal with it... 😕 (At least I think I'm going to for openSUSE and Mageia, because this pain isn't worth it.) |
I can't find the specific reasons this is failing and we need to get containers building again for ELN. Let's merge this and do a build and we'll figure the rest out alongside Rawhide. |
141c1fa
Merged. I'm logging out now. I will do the Fedora build on Monady, unless somebody else does it sooner. |
This change will be in dnf5-5.1.15-2.fc41 which is now building. |
Matches rpm-software-management/dnf5#1309 for DNF 5. Signed-off-by: Stephen Gallagher <[email protected]>
Matches rpm-software-management/dnf5#1309 for DNF 5. Signed-off-by: Stephen Gallagher <[email protected]>
The necessary pieces are not yet in place for RHEL 11 and this results in conflicts.