[virt-tools-list] libosinfo - implemented.

[Arjun Roy]

GObject sounds like it has its benefits, and I'm definitely open to the idea of using it for implementation. But for now, I'd like to make sure the programmer interface itself is acceptable, since we can always change out the implementation of the library itself once the API is decided upon.

I'd go so far as to say that let's figure out API and packaging with the code as is for now, and once that's settled the implementation can be modified whenever convenient.

-Arjun

----- Original Message -----
From: "Daniel P. Berrange" <berrange at redhat.com>
To: "Arjun Roy" <arroy at redhat.com>
Cc: "virt-tools-list" <virt-tools-list at redhat.com>, "Cole Robinson" <crobinso at redhat.com>, "Hugh O. Brock" <hbrock at redhat.com>
Sent: Monday, February 1, 2010 4:48:09 AM GMT -08:00 US/Canada Pacific
Subject: Re: libosinfo - implemented.


I've only had a quick look at the API / code so far, but my first impression
is that the API would majorly benefit by being ported to use GLib / GObject
rather than being built from scratch. In particular this would let you use
a good object system, which your code is clearly trying to achieve. The use
of the structure list / hash table data types & error reporting in GLib would
also bring benefits to this.  Finally, perhaps the most compelling reason is
that with GObject Introspection, we will get the ability to access libosinfo
from multiple other languages for free, without needing to hand write or 
auto generate any extra code.  The only minor downside is the OOM handling
in GLib, but for the scenarios in which libosinfo is to be used I don't
think that this is a problem worth worrying about & is easily outweighed by
the advantages.

Regards,
Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|