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

[spec] Remove /etc/dnf/protected.d/dnf.conf #1991

Merged

Conversation

evan-goode
Copy link
Member

@evan-goode evan-goode commented Sep 20, 2023

Instead, hardcode the protection of dnf in libdnf. This way, DNF 5 will be able to uninstall DNF 4.

Requires: rpm-software-management/libdnf#1621
Related: rpm-software-management/dnf5#906

@inknos
Copy link
Contributor

inknos commented Sep 21, 2023

same note as here then lgtm

@inknos
Copy link
Contributor

inknos commented Sep 21, 2023

Oh actually, since this PR requires rpm-software-management/libdnf#1621, it would be nice to add libdnf's version in the spec file.

I would propose to bump both libdnf and dnf's versions and add the minimum requirements in the spec.

other than this it looks good to me

Instead, hardcode the protection of dnf in libdnf. This way, DNF 5 will
be able to uninstall DNF 4.

= changelog =
msg: Allow dnf to be removed by DNF 5
type: enhancement
Require new version of libdnf so that dnf remains a protected package,
see rpm-software-management/libdnf#1621.
@evan-goode evan-goode force-pushed the evan-goode/self-protection branch from 051fd22 to 87bd305 Compare September 21, 2023 18:13
@evan-goode
Copy link
Member Author

Oh actually, since this PR requires rpm-software-management/libdnf#1621, it would be nice to add libdnf's version in the spec file.

I would propose to bump both libdnf and dnf's versions and add the minimum requirements in the spec.

other than this it looks good to me

Sounds good, bumped the version. @jan-kolarik would you be able to take a look at this and rpm-software-management/libdnf#1621? I'm not sure what the release workflow is like.

Copy link
Member

@jan-kolarik jan-kolarik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks

@inknos inknos merged commit 552e61e into rpm-software-management:master Sep 25, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants