[PATCH] Remove emitCheckFailed(), change it to logger.error(). BZ 1048391

Zdenek Pavlas zpavlas at redhat.com
Tue Jan 7 09:25:07 UTC 2014


> > > -            self.emitLockFailed()
> > > +            self.logger.error()
 
>  Having the user disable all error messages seems unlikely to be a good
> enough hack ...

AIUI, most of the time a failed lock is not an error, but it should not
be ignored... logger.warn() seems to be a good match. The shipped
yum-cron-hourly.conf uses debuglevel = -2 and this *includes* warnings.

> third option that comes to mind is to just close it as WONTFIX.

IMO, as bad as turning off errors (or warnings) is, it should be supported.


More information about the Yum-devel mailing list