Description of problem: Fedora 22 is planning to switch over to dnf from yum as per https://fedoraproject.org/wiki/Changes/ReplaceYumWithDNF Please switch from yum/yum-utils to dnf/dnf-plugins-core. Thanks!
Created attachment 1045877 [details] use dnf instead of yum Here is a patch for system-config-kickstart which replaces yum with dnf.
Applied, thanks for the patch!
Hey, I just applied this and was wondering if you'd seen the following traceback: Traceback (most recent call last): File "src/system-config-kickstart", line 95, in <module> ksGui.run() File "/home/clumens/src/system-config-kickstart/src/kickstartGui.py", line 203, in run self.packages_class = packages.Packages(xml, self.ksHandler, self.progress_window) File "/home/clumens/src/system-config-kickstart/src/packages.py", line 176, in __init__ self.gs.doRefresh() File "/home/clumens/src/system-config-kickstart/src/GroupSelector.py", line 448, in doRefresh self._setupCatchallCategory() File "/home/clumens/src/system-config-kickstart/src/GroupSelector.py", line 430, in _setupCatchallCategory for g in cat.groups: File "/usr/lib/python2.7/site-packages/dnf/comps.py", line 219, in groups return list(self.groups_iter()) File "/usr/lib/python2.7/site-packages/dnf/comps.py", line 215, in groups_iter yield self._build_group(grp_id) File "/usr/lib/python2.7/site-packages/dnf/comps.py", line 210, in _build_group raise ValueError(msg % (grp_id.name, self.id)) ValueError: no group 'clustering' from category 'servers'
I don't remember such a traceback but the patch is half a year old so most likely code has changed since then.
Which just means that I would have had a lot of other dnf-related bugs in s-c-kickstart since then. This doesn't really fill me with optimism about using dnf. Maybe one day it'll become a stable library, but it doesn't seem like we are there yet.
s-c-ks in its current form is probably never going to move to DNF. It's really old code at this point and could use some serious reworking, at which point I will just start with DNF from scratch. Thanks for the patch, though.