[libvirt] [libvirt-glib PATCHv5 7/7] gobject: Add wrapper for virNetworkGetDHCPLeases

Zeeshan Ali (Khattak) zeeshanak at gnome.org
Wed Jul 8 12:13:09 UTC 2015


On Wed, Jul 8, 2015 at 11:11 AM, Christophe Fergeau <cfergeau at redhat.com> wrote:
> On Tue, Jul 07, 2015 at 05:27:39PM +0100, Zeeshan Ali (Khattak) wrote:
>> Hi all,
>>
>> Christophe pointed out that this and the previous patch binds API that
>> was added an year ago:
>>
>> commit:  03e0e79e07622496522609741734c2fdcacb5bf2
>> Author: Nehal J Wani <nehaljw.kkd1 at gmail.com>
>> Date:   Tue Jun 24 02:31:49 2014 +0530
>>
>>     net-dhcp-leases: Implement the public APIs
>>
>>     Introduce 3 new APIs, virNetworkGetDHCPLeases, virNetworkGetDHCPLeasesForMAC
>>     and virNetworkDHCPLeaseFree.
>> ---------------
>>
>> While I think 1 year old is pretty old enough to justify bumping the
>> dep to that version, Christophe thinks its still too soon. I'd hate to
>> go through the trouble of adding ugly #ifdef around these new API but
>> if others (I guess mainly Dan?) agree with Christophe, I can do that.
>
> 1 year being old/not old is not a very useful/convincing argument. What
> is more interesting is what (supported) distros are shipping (f21, el7,
> oldest ubuntu LTS shipping libvirt-glib, debian, ...).

For the record, I don't think upstream development should depend on
when distros decided to upgrade packages and their release cycles. The
important thing is for us to give them enough time and IMO 1 year is
plenty of time. Also keeping in mind that it makes very little sense
to upgrade libvirt-glib and not libvirt since libvirt doesn't break
any ABI/API.

> I'd tend to favour #ifdefs though unless this gets very invasive/ugly ;)

Well we don't agree and that's why I brought it up here. :)

-- 
Regards,

Zeeshan Ali (Khattak)
________________________________________
Befriend GNOME: http://www.gnome.org/friends/




More information about the libvir-list mailing list