[libvirt] [PATCH] complete virterror->virerror name change

Eric Blake eblake at redhat.com
Fri Feb 1 21:06:56 UTC 2013


On 02/01/2013 12:55 PM, Serge Hallyn wrote:
> Quoting Eric Blake (eblake at redhat.com):
>> On 01/30/2013 08:05 PM, Serge Hallyn wrote:
>>> Without these two string changes in generator.py, the
>>> virGetLastError wrapper does not get created in
>>> /usr/share/pyshared/libvirt.py.  Noticed when running
>>> tests with virt-install.
>>>
>>> Signed-off-by: Serge Hallyn <serge.hallyn at ubuntu.com>
>>> ---
>>>  python/generator.py | 4 ++--
>>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> ACK.
>>
>> Bummer - that means the python bindings are broken in the 1.0.2 release.
> 
> D'oh, with my patch, Chuck was also getting the following error:
> 
> chuck at homer:~/work/server/libvirt/libvirt-1.0.2/python/tests$ python error.py 
> Traceback (most recent call last):
>  File "error.py", line 35, in <module>
>   elif errno[0] == libvirt.VIR_ERR_NO_CONNECT or \
> AttributeError: 'module' object has no attribute 'VIR_ERR_NO_CONNECT'
> 
> presumably because include/libvirt/virterror.h still exists, and my
> patch was over-zealous.

Does that mean we need a followup patch that looks for both virerror AND
virterror?

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130201/297b9b58/attachment-0001.sig>


More information about the libvir-list mailing list