On Wed, 2010-11-03 at 17:00 -0400, seth vidal wrote:
> On Wed, 2010-11-03 at 15:29 -0400, James Antill wrote:
> 
> > > +        
> > > +        try:
> > > +            base.load_ts(load_file)
> > > +        except yum.Errors.YumBaseError, e:
> > > +            return 1, [to_unicode(e)]
> > > +        return 2, [_('Transaction loaded from %s with %s members') % 
> > > (load_file, len(base.tsInfo))]
> > 
> >  load_ts to return the new txmbrs, so yum shell dtrt.
> 
> load_ts cannot return the new txmbrs - nor more so than installPkgs()
> can return the new txmbrs.
> 
> What do you mean here?

 Oh, just the len(base.tsInfo) is the number of all the txmbr's, not
just the ones added with this call.
 .installPkgs() could return them, it just doesn't.

 Thinking about it more the message isn't displayed when we are in
shell, so it's not big deal to leave it as it.

_______________________________________________
Yum-devel mailing list
Yum-devel@lists.baseurl.org
http://lists.baseurl.org/mailman/listinfo/yum-devel

Reply via email to