On Wed, 2013-12-11 at 03:57 -0500, Zdenek Pavlas wrote: > > > - if self.conf.group_command == 'objects' and ninstalled: > > > - # Show new pkgs. that are installed via. a group. > > > > Did you mean to remove this check? > > I purposely did that as without object groups it seemed a bit weird to > > say that the packages were installed because of X but then we'd forget > > about it later. I guess people might still find it useful though. > > Uh, sorry.. should have added more comments to the commit. > Since this runs as a method of StdioEmitter, self.tsInfo > and self.conf access triggers an error.
Ahh, I see. I tried following it back and it looked like self.tsInfo was valid :(. > Making the code unconditional is easier and maybe better > than passing the conf object or a flag from the caller. Ok, I can live with it until someone complains :). Can you do the two changes in different commits though (change to not access self.tsInfo and self.conf, and then any cleanup in another) ... also change yum to be the same, so we can copy&paste fixes between them (or create some functions somewhere and call those from yum-cron :). _______________________________________________ Yum-devel mailing list Yum-devel@lists.baseurl.org http://lists.baseurl.org/mailman/listinfo/yum-devel