Re: [PATCH V3 1/2] libxl: drop support for Xen < 4.6

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9/17/18 2:59 AM, Andrea Bolognani wrote:
On Tue, 2018-09-11 at 16:38 -0600, Jim Fehlig wrote:
[...]
Since Xen 4.6 contains a pkgconfig file, drop the now unused code
that falls back to using LIBVIRT_CHECK_LIB in the absence of
pkgconfig file.

[...]
-  dnl pkgconfig file not found, fallback to lib probe
-  if test "x$with_libxl" = "xno" ; then
-    with_libxl="$old_with_libxl"
-
-    dnl LIBXL_API_VERSION 4.4.0 introduced a new parameter to
-    dnl libxl_domain_create_restore for specifying restore parameters.
-    dnl The libxl driver will make use of this new parameter for specifying
-    dnl the Xen migration stream version. Specify LIBXL_API_VERSION to trigger
-    dnl an error if there is too old xenlight
-    libxlold_CFLAGS="$CFLAGS"
-    CFLAGS="$CFLAGS $LIBXL_API_VERSION"
-    LIBVIRT_CHECK_LIB([LIBXL], [xenlight], [libxl_ctx_alloc], [libxl.h], [fail="1"])
-    CFLAGS="$libxlold_CFLAGS"
-
-    if test $fail = 1; then
-      AC_MSG_ERROR([You must install the libxl Library from Xen >= 4.4 to compile libxenlight driver with -lxl])
-    fi
-  fi

This broke CI builds:

   https://ci.centos.org/view/libvirt/job/libvirt-rpm/

It's not libvirt's fault, but rather an issue with Xen packaging
which I've reported accordingly:

   https://bugzilla.redhat.com/show_bug.cgi?id=1629643

Thanks, I've added myself to the bug cc list.

Depending on how long it takes to get that fixed in Fedora, we
might want to temporarily reintroduce the fallback path...

Ok. What is your estimate of "long" in this case? :-) By the end of the day? End of week? Before freeze for next release?

Regards,
Jim

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



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux