[rhel6-branch][PATCH] Check if selected repos have the core package group (#901515)

Martin Kolman mkolman at redhat.com
Tue Jul 16 06:27:47 UTC 2013


On Mon, 2013-07-15 at 15:54 -0700, Brian C. Lane wrote:
> On Mon, Jul 15, 2013 at 08:26:06PM +0200, Martin Kolman wrote:
> > Check if the selected repositories contain the "core" package group,
> > which is a requirement for successful installation.
> > If the core group is not available, show warning dialog if next is clicked.
> > If core is available, let the installation proceed as usual.
> > 
> > Signed-off-by: Martin Kolman <mkolman at redhat.com>
> > ---
> >  iw/task_gui.py | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> > 
> > diff --git a/iw/task_gui.py b/iw/task_gui.py
> > index dc574d7..5cacc67 100644
> > --- a/iw/task_gui.py
> > +++ b/iw/task_gui.py
> > @@ -523,6 +523,14 @@ class TaskWindow(InstallWindow):
> >          # through the UI.
> >          self.backend.ayum.comps = None
> >  
> > +        # check if the selected repositories contain the core package group,
> > +        # which is required for installation
> > +        if not self.backend.ayum.comps.has_group("core"):
> > +            self.anaconda.intf.messageWindow(_("Core group missing in selected repos"),
> > +            _("You must select at least one software repository containing the"
> > +              " core package group."))
> > +            raise gui.StayOnScreen
> > +
> >          tasks = self.xml.get_widget("taskList").get_model()
> >          for (cb, task, grps) in filter(lambda x: not x[0], tasks):
> >              map(lambda g: setattr(self.backend.ayum.comps.return_group(g),
> > -- 
> > 1.8.3.1
> 
> This looks good for GUI. Shouldn't we also do the check for text?
Good point, I'll check if the bug is also present in TUI.
> 
> _______________________________________________
> anaconda-patches mailing list
> anaconda-patches at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches




More information about the anaconda-patches mailing list