[virt-tools-list] [libosinfo PATCHv3 10/12] Add osinfo_install_config_new_for_script

Zeeshan Ali (Khattak) zeeshanak at gnome.org
Thu Dec 13 16:06:25 UTC 2012


On Thu, Dec 13, 2012 at 4:34 PM, Christophe Fergeau <cfergeau at redhat.com> 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 at redhat.com> 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




More information about the virt-tools-list mailing list