[PATCH] Remove emitCheckFailed(), change it to logger.error(). BZ 1048391
james.antill at redhat.com
Tue Jan 7 22:13:52 UTC 2014
On Tue, 2014-01-07 at 04:25 -0500, Zdenek Pavlas wrote:
> > > > - 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.
I bet most people would be happy if we could do something like ignore
it the first time, _esp._ for the hourly runs. And, yeh, warning seems
fine to me.
> > 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.
Yeh, but I'm also not dying to answer "how can I turn the lock message
off" with "turn all warnings off". That's a pretty big hammer.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 190 bytes
Desc: This is a digitally signed message part
More information about the Yum-devel