-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Add fix for package running dnf5 on fedora #9520
base: main
Are you sure you want to change the base?
Conversation
Fedora 41 introduced dnf5 which has deprecated the use of the flags '-d' and '-e'. As a workaround, this change will exlude the flags when running on fedora >= 41. Signed-off-by: Lars Haugan <[email protected]>
Can one of the admins verify this patch? |
I just tested the PR, works great. Up next:
|
If I recall correctly, for dnf4, the old options were already deprecated. I don't know if dnf3 is still in use somewhere or how old it is, rocky linux 8 already uses dnf4. centos 7 also uses dnf4. Perhaps we can do away with the conditionals altogether? from the centos 7 dnf manpage:
|
@QWolfp3 how can I help in moving this PR along? |
IMHO, we need a simple PR which does away with the deprecated settings. Those are settings from the RHEL6 era, which is no longer supported. |
@Stricken1670 I created a branch for removing the flags: https://github.com/puppetlabs/puppet/compare/main...larhauga:puppet:fix/main/dnf5-fedora41-remove-deprecated-flags?expand=1 It would be much cleaner and the best option if you think that we can break compatability here. I did not manage to find out where in the codebase the yum groupinstall comes from. |
The calls to |
@larhauga |
Hi,
on fedora 41 we experience the following errors due to the new dnf version, dnf5 (https://docs.fedoraproject.org/en-US/fedora/latest/release-notes/sysadmin/#dnf-5).
The flags '-d' and '-e' have as a workaround been removed for newer versions of fedora.
This fixes #9506