-
Notifications
You must be signed in to change notification settings - Fork 10
Add fix for package running dnf5 on fedora #20
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
Conversation
Fedora 41 introduced dnf5 which has deprecated the use of the flags '-d' and '-e'. This change removes the options Signed-off-by: Lars Haugan <[email protected]>
I should note that the |
Are they removed with no replacement or should we find an alternate way to get the same functionality? |
Here is the original issue: |
Agree I don't think we need all the code in the Puppet version. I'm not sure if we should use -q or not. -q is recommended in the dnf docs, but I'm not sure if that gets rid of things we try to parse. Since we are no longer supporting anything pre-el7, I think this should be fine. |
I don't see anything being parsed from the DNF output. If the installation fails the output is displayed to the user, with -q you see enough details, without -q you will see too many details IMHO. OTOH, I do not want to overcomplicate this excellent PR. We can add the |
We at least parse the output here: https://github.com/OpenVoxProject/puppet/pull/20/files#diff-f1faf8e1a8d06a2175e65b961a10236d362326009dd8d0bd4e93898cc2857f39R313 |
Ah, I missed that. That's no longer necessary. This is on fedora 41:
And on centos 7
|
Fedora 41 introduced dnf5 which has deprecated the use of the flags
-d
and-e
. This change removes the options.Credit to @larhauga in puppetlabs/puppet#9536
Closes #19