Re: [PATCH 1/2] xen: add error handling to UUID parsing
|[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]|
On 10/07/2011 04:18 AM, Guido Günther wrote:
> >>ACK to the concept, once we decide on the correct error code.> > I think we've decided.(Hopefully) final version attached. I've left the first error as INTERNAL_ERROR since this it's there do detect false function parameters.
Yes, you got that part right.
Subject: [PATCH] xen: add error handling to UUID parsing otherwise a missing UUID in a domain config just shows: error: An error occured, but the cause is unknown
s/occured/occurred/I don't see that typo in the source code for virterror, so I suspect you were re-typing instead of copy-and-paste. (That said, I'll be pushing a trivial patch to fix up the places that _do_ have a typo.)
Now we have: error: configuration file syntax error: config value uuid was missing
ACK. -- Eric Blake eblake@xxxxxxxxxx +1-801-349-2682 Libvirt virtualization library http://libvirt.org -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
[Virt Tools] [Libvirt Users] [Fedora Users] [Fedora Legacy] [Fedora Maintainers] [Fedora Desktop] [Fedora SELinux] [Big List of Linux Books] [Yosemite News] [Yosemite Photos] [KDE Users] [Fedora Tools]