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

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

 



On Wed, Jul 8, 2015 at 11:11 AM, Christophe Fergeau <cfergeau@xxxxxxxxxx> 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@xxxxxxxxx>
>> 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/

--
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]