Re: [PATCH] Allow displaying groups that only contain conditional packages (#475239).

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Looks ok to me.

Chris Lumens wrote:
This also allows selecting these groups in the UI, though if the group has no
optional packages you will not be able to pull up the optional package
selection dialog.  In effect, a conditional-only group will act like a
mandatory-only group as far as the UI is concerned.
---
 yuminstall.py |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/yuminstall.py b/yuminstall.py
index 604da7b..3e0b1e2 100644
--- a/yuminstall.py
+++ b/yuminstall.py
@@ -1094,7 +1094,8 @@ class AnacondaYum(YumSorter):
                 return True
         if len(grp.mandatory_packages) > 0:
             return False
-        for pkg in grp.default_packages.keys() + grp.optional_packages.keys():
+        for pkg in grp.default_packages.keys() + grp.optional_packages.keys() + \
+                   grp.conditional_packages.keys():
             if self._pkgExists(pkg):
                 return True
         return False

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux