[Libvir] [PATCH] Tidy up the error messages produced by test driver

Richard W.M. Jones rjones at redhat.com
Wed Apr 18 13:35:26 UTC 2007


Daniel Veillard wrote:
> On Wed, Apr 18, 2007 at 10:38:17AM +0100, Richard W.M. Jones wrote:
>> This minor patch:
>>
>> * Introduces a VIR_FROM_TEST class of errors and makes errors generated 
>> by test come from that class.
>>
>> * Provides a reasonable error message if the user forgets to supply a 
>> path.  Also stops the driver from trying to open ("/").  Example:
>>      $ src/virsh -c test:/// capabilities
>>      libvir: Test error : invalid argument in testOpen: supply a
>>      path or use test:///default
>>      lt-virsh: error: failed to connect to the hypervisor
>>
>> * Arranges that Xen-unified appears last in the list of drivers, because 
>> qemu and test have well-defined URI syntaxes so can be checked first.
>>
>> 'make check' passes all tests with this patch.
> 
>   looks fine by me, but I prefer fully parenthesized tests in conditions :-)

This is committed to CVS.

Rich.

-- 
Emerging Technologies, Red Hat  http://et.redhat.com/~rjones/
64 Baker Street, London, W1U 7DF     Mobile: +44 7866 314 421

Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod
Street, Windsor, Berkshire, SL4 1TE, United Kingdom.
Registered in England and Wales under Company Registration No. 3798903
Directors: Michael Cunningham (USA), Charlie Peters (USA) and David
Owens (Ireland)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3237 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20070418/2e3b97f8/attachment-0001.bin>


More information about the libvir-list mailing list