Re: [libosinfo PATCHv3 10/12] Add osinfo_install_config_new_for_script

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

 



On Thu, Dec 13, 2012 at 4:34 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> On Thu, Dec 13, 2012 at 03:42:16PM +0200, Zeeshan Ali (Khattak) wrote:
>> On Thu, Dec 13, 2012 at 2:06 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
>> > as a nice improvement to the codebase even if there were not the datamap
>> > changes on top of it.
>>
>> Thanks for the explanation and I would agree with you to everything
>> you said here but this kinda breaks API/ABI cause user is now expected
>> to use a different _new(). I'll repeat the question I asked before:
>> How does the app know which _new() to use? Now that we'll be expecting
>> OS-specific values in scripts, any existing code (or even future code
>> that uses the existing _new()) will still pass us OS-independent
>> config that did not see the transformation.
>
> Ah, good point about the ABI break, I'll have to think about this. In the
> mean time, can we get the 3 first patches in this series in? I think this
> all that is needed for the
> osinfo_db_identify_media/osinfo_media_get_languages series, which would be
> nice to land...

Yeah.

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list


[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux