On Thu, Feb 18, 2010 at 06:14:11PM +0000, Sharadha Prabhakar (3P) wrote:
> This is a patch to add XenAPI driver support for libvirt version 0.7.6. 
> XenAPI can be used against XenCloud platform and
> managed through virsh and virt-manger. This patch supports domain related 
> APIs in libvirt. It is possible to get domain information,
> list active and inactive domains, get Domain XML configuration and 
> Start/stop/pause/shutdown/destroy VMs.
> There will be more patches after this review to support more libvirt APIs and 
> add remote storage support to XenAPI.
> In order to run this patch you would require libxenserver library.

  thanks for the patch, just started to look at it.

  I assume that by libxenserver you mean the code to be added under src
in a following patch, right ?

[...]
> +endif
> +#libvirt_driver_xenapi_la_LIBADD = $(XENAPI_LIBS)
> +libvirt_driver_xenapi_la_CFLAGS = $(XEN_CFLAGS) \
> +                                  $(shell xml2-config --cflags) \
> +                                  $(shell curl-config --cflags)
> +libvirt_driver_xenapi_la_LDFLAGS = -...@top_srcdir@/../libxenserver/ 
> -lxenserver -g $(shell xml2-config --libs) \
> +                                  $(shell curl-config --libs)

  Hum, libxml2 is already fully detected in configure.ac, use the
LIBXML_* variables instead of trying to exec xml2-config.

  Similary libcurl i looked at in configure.ac for ESX, this need
to be modified to be checked more generally and the LIBCURL_*
variables need to be used here too.

  thanks,

Daniel


-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
dan...@veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to