[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] util: use string libvirt to prefix error message instead of libvir



On Mon, Mar 04, 2013 at 09:50:23AM -0500, Dave Allan wrote:
> On Mon, Mar 04, 2013 at 08:54:10PM +0800, Guannan Ren wrote:
> > BZ:https://bugzilla.redhat.com/show_bug.cgi?id=912021
> > Without error handler set, virDefaultErrorFunc will be called, the
> > error message is prefixed with "libvir:". It become a little better
> > by using prefix "libvirt:" when working with upper application.
> 
> My concern is that making this change could break scripts that depend
> on the old spelling.  Guannan mentioned to me that this error doesn't
> arise when using virsh.  I don't have a strong opinion either way, but
> I think if there's no danger of breaking existing code that depends on
> the spelling then we should fix this, otherwise leave it as is.

The reason it doesn't hurt virsh is that it disabled the default
error reporting function. It reports errors directly itself. In
fact most apps end up disabling the default error reporting
function & doing their own error reporting to stderr, because
the default is not flexible enough.

/me wishes we didn't ever have it in the first place - libraries
have no business printing anything to stderr, without explicit
app opt-in.

I think it would be ok to rename this though. Apps relying on
screen-scraping this text are doomed - we've never considered
error message text to be part of the stable interface, nor
even error codes - we'll often change codes to better ones over
time.

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]