revisor/base.py | 27 +++++++++++++++++++--------
1 file changed, 19 insertions(+), 8 deletions(-)
New commits:
commit fbf6ff3d25bf2382a93673fc8219cf0ce7eb317c
Author: Jeroen van Meeuwen (Fedora Unity) <kanarip(a)fedoraunity.org>
Date: Thu Dec 11 16:58:10 2008 +0100
Fix the ksgroup.name vs. ksgroup being a name issue
diff --git a/revisor/base.py b/revisor/base.py
index 0848e04..00df975 100644
--- a/revisor/base.py
+++ b/revisor/base.py
@@ -474,22 +474,33 @@ class RevisorBase:
## Add packages from Groups in ksdata
##
for ksgroup in groupList:
- if not self.cfg.yumobj.comps.has_group(ksgroup.name):
- self.log.warning(_("No such group %s") % ksgroup.name )
+ if hasattr(ksgroup,"name"):
+ group_name = ksgroup.name
+ else:
+ group_name = ksgroup
+
+ if not self.cfg.yumobj.comps.has_group(group_name):
+ self.log.warning(_("No such group %s") % group_name )
continue
else:
- self.log.debug(_("Found group: %s") % ksgroup.name)
+ self.log.debug(_("Found group: %s") % group_name)
# Get group object from comps
- grp = self.cfg.yumobj.comps.return_group(ksgroup.name)
+ grp = self.cfg.yumobj.comps.return_group(group_name)
txmbrs_used = self.cfg.yumobj.selectGroup(grp.name)
- if ksgroup.include >= kickstart.constants.GROUP_REQUIRED:
+ try:
+ if ksgroup.include >= kickstart.constants.GROUP_REQUIRED:
+ pkglist.extend(grp.mandatory_packages.keys())
+ if ksgroup.include >= kickstart.constants.GROUP_DEFAULT:
+ pkglist.extend(grp.default_packages.keys())
+ if ksgroup.include >= kickstart.constants.GROUP_ALL:
+ pkglist.extend(grp.optional_packages.keys())
+ except:
+ # If such fails, just include the mandatory and defaults
pkglist.extend(grp.mandatory_packages.keys())
- if ksgroup.include >= kickstart.constants.GROUP_DEFAULT:
pkglist.extend(grp.default_packages.keys())
- if ksgroup.include >= kickstart.constants.GROUP_ALL:
- pkglist.extend(grp.optional_packages.keys())
+
##
## Pull the conditional packages in
Show replies by date