Re: [libvirt] [PATCH 1/2] (Updated & fixed) OpenNebula driver, libvirt-0.6.2
On Tue, May 26, 2009 at 04:26:36PM +0200, Ruben S. Montero wrote:
Hi,
Probably the easiest way to solve this is to get rid to of the
OpenNebula client library. So we do not have the license issue and, as
Daniel said, we do not have to include the C++ dependencies.
We'll try to send the patch ASAP.
Okay, thanks !
Ideally by Friday since I will release 0.6.4 then, if we could have this
fixed before the realease that would be great :-)
Daniel
On Tue, May 26, 2009 at 3:16 PM, Daniel P. Berrange
<berrange(a)redhat.com> wrote:
[...]
>> 6. talk XML RPC directly to the server and not use the
client APIs
>
> After looking at the scope of the APIs, I'm inclined to say we should
> take option 6. The current impl
>
> - libvirt one driver -> one C client
> - one C client is a wrapper around one C++ client
> - one C++ client talks to XMLRPC-C C++ library
> - XMLRPC-C C++ library is a wrapper around XMLRPC-C library
>
> So, not only do we currently have a license problem, but we're pulling in the
> whole C++ runtime for no particularly good reason or benefit. The APIs being
> invoked from the libvirt ONE driver are pretty simple - even the C++ client
> library is a merely couple of 100 lines of code. So I think its pretty easy
> to just have the libvirt one driver use xmlrpc C library directly.
>
> Regards,
> Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel(a)veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/