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

Support comps #107

Closed
cgwalters opened this issue Apr 27, 2016 · 13 comments
Closed

Support comps #107

cgwalters opened this issue Apr 27, 2016 · 13 comments
Assignees
Labels

Comments

@cgwalters
Copy link
Collaborator

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

@hughsie
Copy link
Contributor

hughsie commented Apr 27, 2016

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

@cgwalters
Copy link
Collaborator Author

@ignatenkobrain
Copy link
Contributor

I think we should consider merging libcomps into libhif.

@Conan-Kudo
Copy link
Member

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).

@jsilhan
Copy link
Contributor

jsilhan commented Apr 29, 2016

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

@cgwalters
Copy link
Collaborator Author

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

@Conan-Kudo
Copy link
Member

Conan-Kudo commented May 1, 2016

@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.

@cgwalters
Copy link
Collaborator Author

cgwalters added a commit to cgwalters/libdnf that referenced this issue Jun 6, 2016
There's a request from fedora releng to support parsing comps groups
for rpm-ostree, this is a RFC/WIP patch for it.  I think the main
architectural questions are:

 - Use @ like yum to denote group names?
 - Add them to the depsolver, but do not return them to the rest
   of the architecture.  This may make it harder to port dnf,
   since I think it currently has persistent groups enabled?

Related: rpm-software-management#107
@bam80
Copy link

bam80 commented May 9, 2020

Just wanted to clarify the status.

@Conan-Kudo
Copy link
Member

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.

@j-mracek
Copy link
Contributor

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.

@dmach
Copy link

dmach commented May 11, 2020

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.

@pkratoch pkratoch self-assigned this Jan 20, 2021
@pkratoch pkratoch added the dnf5 label Jan 20, 2021
@j-mracek
Copy link
Contributor

j-mracek commented Mar 2, 2023

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 j-mracek closed this as completed Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

9 participants