[libvirt] FW: [PATCH 1/4] Addition of XenAPI support to libvirt

[Daniel Veillard]

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 = -L at 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/
daniel at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/