libdnf icon indicating copy to clipboard operation
libdnf copied to clipboard

Support comps

Open cgwalters opened this issue 8 years ago • 12 comments

I think previously this was deemed somewhat ugly because libcomps is a non-glib library...should we consider folding it in directly into libhif and adding the glib dep? (Would the authors agree?)

But it's likely for rpm-ostree at least we'll need to support comps for https://mail.gnome.org/archives/ostree-list/2016-April/msg00020.html https://mail.gnome.org/archives/ostree-list/2016-April/msg00010.html

cgwalters avatar Apr 27 '16 13:04 cgwalters

I'm really not keen on the design of libcomps at all. I'm totally for porting and folding into libhif.

hughsie avatar Apr 27 '16 13:04 hughsie

https://github.com/rpm-software-management/libcomps/issues/22

cgwalters avatar Apr 27 '16 21:04 cgwalters

I think we should consider merging libcomps into libhif.

ignatenkobrain avatar Apr 28 '16 07:04 ignatenkobrain

I'm all for having the comps functionality as part of libhif, but I think we would probably need some sort of Comps Python interface preserved for portability (like we've done for the legacy hawkey Python bindings).

Conan-Kudo avatar Apr 28 '16 11:04 Conan-Kudo

I am for merging libcomps although at first we should merge librepo. Isn't libsolv comps support just enough for rpm-ostree purposes?

jsilhan avatar Apr 29 '16 10:04 jsilhan

I was totally unaware libsolv had comps support. I'll look at that.

cgwalters avatar May 01 '16 13:05 cgwalters

@cgwalters According to @mlschroe, it's somewhat rudimentary (though I forget exactly what the limits of the built-in comps support are). As of libsolv 0.6.20 in Fedora, the comps support is enabled. Extending libsolv's comps support (as needed) and using that in libhif is probably a much more solid path.

Conan-Kudo avatar May 01 '16 13:05 Conan-Kudo

https://bugzilla.redhat.com/show_bug.cgi?id=1340556

cgwalters avatar Jun 01 '16 14:06 cgwalters

Just wanted to clarify the status.

bam80 avatar May 09 '20 15:05 bam80

This is definitely going to be supported as part of the dnf5 work. @dmach or @j-mracek would have a better idea of when that will land.

Conan-Kudo avatar May 09 '20 15:05 Conan-Kudo

I am not 100 % sure what exactly is requested. dnf5 work means to move a lot of functionality from dnf to libdnf including support of comps, and adding of query functionality for comps. Due to our limitation we cannot reimplement libcomps in libdnf. May be in far far future. dnf5 will be written in C++ and we will provide bindings for c, python and so on. We have no plan to support glib macros, therefore glib auto-free functions will require to provide a destructor for proper functionality.

j-mracek avatar May 10 '20 17:05 j-mracek

Let me clarify - the new libdnf major version (currently dnf-5-devel branch) will support working with comps groups. It's going to use libcomps as a backend for reading comps information, but the libcomps internals will not be exposed, only a high-level libdnf API will. No glib support is planned as we're moving libdnf away from glib to C++17.

dmach avatar May 11 '20 05:05 dmach

I have a good news - comps support was implementd in DNF5 (available in Fedora38) project. We do not have a plan to extent the LIBDNF functionality, therefore it will be replaced by DNF5 (Fedora 39). I am closing the issue.

j-mracek avatar Mar 02 '23 14:03 j-mracek