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@xxxxxxxxxxxx | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/ -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list