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

Suggests DNF5 installation on 38+ again #1220

Merged
merged 2 commits into from
Sep 14, 2023

Conversation

praiskup
Copy link
Member

Actually, having DNF5 on Fedora 38 Mock host is still a good idea for the future when we'll be building Fedora 41 chroots.

This reverts commit 8cca2d0.

Actually, having DNF5 on Fedora 38 Mock **host** is still a good idea
for the future when we'll be building Fedora 41 chroots.

This reverts commit 8cca2d0.
@praiskup
Copy link
Member Author

praiskup commented Sep 13, 2023

FYI @evan-goode @j-mracek - mostly related to our today discussion that we should propose building of Fedora 41 chroots with dnf5 rather soon.

@Conan-Kudo
Copy link
Member

Suggests doesn't do anything. You need to use Recommends instead if you want it to actually get installed.

@praiskup
Copy link
Member Author

With the bootstrap image feature ON by default, it really isn't worth more than just Suggests?

@Conan-Kudo
Copy link
Member

We don't have it on by default universally.

@praiskup
Copy link
Member Author

That's true, the question is what to do about it. Do you have a proposal? :)

My understanding is that users have to decide whether they prefer (a) bootstrap image, (b) any package manager (that is actually able to install bootstrap, aka bootstrap image off) or (c) a particular package manager version on the host. Pushing DNF5 installation on user's hosts doesn't seem so necessary considering that we already kinda push for (a). The image feature needs to settle down, but long term it is going to be the generally preferred variant, I believe.

@Conan-Kudo
Copy link
Member

I think we should use Recommends for the package managers. If they don't want it, they can uninstall it and it won't come back again.

@praiskup praiskup merged commit ead1315 into rpm-software-management:main Sep 14, 2023
@decathorpe
Copy link

Meh, this is really annoying. I'll need to add exclude=dnf5,yum to my dnf configuration because of this.

@j-mracek
Copy link

j-mracek commented Sep 18, 2023

Meh, this is really annoying. I'll need to add exclude=dnf5,yum to my dnf configuration because of this.

May I ask you what kind of problem you have with DNF5? The problems can be resolved.

@decathorpe
Copy link

I don't have a problem with dnf5, but I don't like the kind of dependency bloat changes like these cause. I already have exclude=podman,toolbox in my dnf.conf, and now it's exclude=podman,toolbox,dnf5,yum ...

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.

5 participants