Re: [PATCH 08/10] Select base-x and fonts when the user selects a non-base desktop.

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

 



Chris Lumens (clumens@xxxxxxxxxx) said: 
> > diff --git a/pyanaconda/ui/gui/spokes/software.py b/pyanaconda/ui/gui/spokes/software.py
> > index 4299de6..adbcea3 100644
> > --- a/pyanaconda/ui/gui/spokes/software.py
> > +++ b/pyanaconda/ui/gui/spokes/software.py
> > @@ -61,6 +61,17 @@ class SoftwareSelectionSpoke(NormalSpoke):
> >          for group in self.selectedGroups:
> >              self.payload.selectGroup(group)
> >  
> > +        # select some stuff people will want with their desktop
> > +        if row[2] != "base":
> > +            groups = ['base-x', 'fonts']#, 'editors', 'graphical-internet',
> > +                      #'hardware-support', 'input-methods', 'java', 'office',
> > +                      #'printing', 'sound-and-video']
> > +        else:
> > +            groups = ['text-internet']
> > +
> > +        for group in [g for g in groups if g not in self.excludedGroups]:
> > +            self.payload.selectGroup(group)
> > +
> >      @property
> >      def completed(self):
> >          return self._get_selected_desktop() is not None
> 
> I understand why we need to do something like this, but I really dislike
> that we have to keep around a list of extra groups still.  I don't
> suppose we are likely to see any changes in package requirements that
> will make this easier on us.  So I guess we are stuck with it.
> 
> For the base case, I don't think we want to include text-internet
> though.  Don't we want that to be a minimal install?

Oops, deleted the first thread without looking closely.

Why is this in anaconda at all?

Bill

_______________________________________________
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