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.

Sign up for GitHub

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

Jump to bottom

Convert to libdnf plugin #1

Open
malmond77 opened this issue Jan 19, 2021 · 2 comments
Open

Convert to libdnf plugin #1

malmond77 opened this issue Jan 19, 2021 · 2 comments
Assignees
@malmond77

Comments

@malmond77
Copy link
Contributor

The Python based plugin system is well established and documented. The downside is that it doesn't work with packagekit, or other tooling that directly interacts with libdnf (including dnf5 / microdnf)

I was originally going to supersede this code and project by doing a PR against libdnf. After more investigation I've found this isn't the right approach. While there is an example plugin in libdnf, it's not easily built. The expectation is that all libdnf plugins will be completely separate packages. To build said packages, you need libdnf-devel, but that's not provided on RHEL / CentOS today making this overly complex. You also need a distinct code base, so this repo continues to have purpose.

As the deadlines for Fedora 34 are looming, I will opt to do the following in the short term:

  1. Establish a new package in rawhide called 'dnf-plugin-cow' with this code
  2. Ship that. It will work with dnf as expected, and remains a limited, opt-in new feature for Fedora 34
  3. Later I or someone else comes back around to this project, and re-implements it as a libdnf plugin. I think it's generally okay to call this project and package dnf-plugin-cow even if should really be called libdnf-plugin-cow. We can rename later if it's important.
@malmond77 malmond77 self-assigned this Jan 19, 2021
malmond77 added a commit that referenced this issue Jan 20, 2021
@malmond77
Renamed package (again) to dnf-plugin-cow
b2117c8 
Using sub-package for python3 based DNF plugin. The goal is to
eventually replace this with libdnf-plugin-cow. See
#1
for rationale

Use Recommends instead of Requires to allow the package to be installed
without complementary changes for rpm and librepo. This is intended to
be temporary until Fedora Rawhide includes the changes.

Fixed the date in an old changelog. Probably was working past midnight
and got the day of week/date mixed up.
@geraldosimiao
Copy link

How about this now that dnf is going to be the default at F41? rpm-software-management/dnf5#389 (comment)

@davide125
Copy link
Contributor

Yes, this is still planned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@malmond77 malmond77

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@davide125 @malmond77 @geraldosimiao