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/