[Yum-devel] [PATCH 4/4] Add --assumeno which does the opposite of --assumeyes.

[James Antill]

On Thu, 2011-07-14 at 14:21 -0400, seth vidal wrote:
> On Thu, 2011-07-14 at 14:19 -0400, James Antill wrote:
> > On Thu, 2011-07-14 at 13:18 -0400, seth vidal wrote:
> > 
> > > > @@ -1535,7 +1535,10 @@ class YumOptionParser(OptionParser):
> > > >                  
> > > >              # Handle remaining options
> > > >              if opts.assumeyes:
> > > > -                self.base.conf.assumeyes =1
> > > > +                self.base.conf.assumeyes = 1
> > > > +            if opts.assumeno:
> > > > +                self.base.conf.assumeno  = 1
> > > > +                self.base.conf.assumeyes = 0
> > > >  
> > > >              #  Instead of going cache-only for a non-root user, try to use a
> > > >              # user writable cachedir. If that fails fall back to cache-only.
> > [...]
> > > looks like assumeyes wins when both are set?
> > 
> >  Yeh, when they are both set in the config. file. When "--assumeyes and
> > --assumeno" are both used, then assumeno wins (due to the above).
> > 
> 
> does that feel.... inconsistent to you?

 Bah ... consistency :)
 Fixed so that they both act the same (made assumeno override assumeyes
in both cases).