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

James Antill 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...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part
URL: <http://lists.baseurl.org/pipermail/yum-devel/attachments/20140107/ab867761/attachment.asc>


More information about the Yum-devel mailing list