Re: [libvirt] [PATCH] Fix generating python bindings

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

 



Daniel P. Berrange wrote:
> On Thu, Feb 12, 2009 at 02:37:46PM -0500, Cole Robinson wrote:
>> The recent API additions virSaveLastError and virFreeError upset the
>> python bindings generator. Add a rule to explicitly skip them for the
>> bindings, since (as far as I can tell) the libvirtError class provides
>> everything we need.
> 
> Correct, this is not required for python layer, since we turn the
> thread local error straight into an exception object the moment
> we get it.
> 
>> diff --git a/python/generator.py b/python/generator.py
>> index 9c71c05..0e8cca7 100755
>> --- a/python/generator.py
>> +++ b/python/generator.py
>> @@ -344,6 +344,8 @@ skip_function = (
>>      'virDefaultErrorFunc', # Python virErrorFuncHandler impl calls this from C
>>      'virConnectDomainEventRegister',   # overridden in virConnect.py
>>      'virConnectDomainEventDeregister', # overridden in virConnect.py
>> +    'virSaveLastError', # We have our own python error wrapper
>> +    'virFreeError', # Only needed if we use virSaveLastError
>>  )
> 
> ACK
> 
> Daniel

Pushed now.

Thanks,
Cole

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]