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

[seth vidal]

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?

-sv