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