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

Packaging changes for DNF => DNF5 upgrade path #1924

Merged
merged 3 commits into from
May 25, 2023

Conversation

evan-goode
Copy link
Member

@evan-goode evan-goode commented Apr 25, 2023

For the DNF => DNF5 upgrade path in Fedora 39. Part of rpm-software-management/dnf5#469.

See also rpm-software-management/libdnf#1599.

It seems that dnf and yum can't be marked as protected even if DNF5 is still going to provide them.

EDIT: The change to unprotect the dnf package would happen before the other changes, so it has been moved to #1926.

dnf.spec Outdated Show resolved Hide resolved
@evan-goode
Copy link
Member Author

This should be ready to go. This PR

  • removes ownership of /etc/dnf/dnf.conf, /etc/dnf/vars, and /etc/dnf/protected.d from dnf-data
  • makes dnf-data require libdnf5, the new owner of /etc/dnf/dnf.conf. Any packages that require the DNF configuration structure should move to depend on libdnf5 instead of dnf-data, but adding this requirement will stop them from breaking immediately.
  • makes dnf-automatic require python3-dnf instead of dnf---it doesn't need the /usr/bin/dnfbinary, just the DNF 4 Python API.

We need to merge this PR and release a new version of DNF before we can merge rpm-software-management/dnf5#469 since the updated DNF 5 spec file need to know the last version number of dnf-data that owns /etc/dnf/dnf.conf.

dnf.spec Outdated Show resolved Hide resolved
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!

@jan-kolarik jan-kolarik merged commit 9aa063d into rpm-software-management:master May 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.

None yet

2 participants