[libvirt] [PATCH 60/89] m4/virt-libxl: use LIBVIRT_CHECK_(PKG|LIB) macros
Jiri Denemark
jdenemar at redhat.com
Wed Dec 21 09:45:07 UTC 2016
On Fri, Dec 16, 2016 at 10:11:28 +0100, Pavel Hrdina wrote:
> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
> ---
> m4/virt-driver-libxl.m4 | 43 +++++++++++++------------------------------
> 1 file changed, 13 insertions(+), 30 deletions(-)
>
> diff --git a/m4/virt-driver-libxl.m4 b/m4/virt-driver-libxl.m4
> index 8382dc0ec4..2fdeaa506e 100644
> --- a/m4/virt-driver-libxl.m4
> +++ b/m4/virt-driver-libxl.m4
> @@ -31,30 +31,19 @@ AC_DEFUN([LIBVIRT_DRIVER_CHECK_LIBXL], [
>
> dnl search for libxl, aka libxenlight
> dnl Xen > 4.5 introduced a pkgconfig file, check for it first
> - fail=0
> - if test "$with_libxl" != "no" ; then
> - PKG_CHECK_MODULES([LIBXL], [xenlight], [
> - LIBXL_FIRMWARE_DIR=`$PKG_CONFIG --variable xenfirmwaredir xenlight`
> - LIBXL_EXECBIN_DIR=`$PKG_CONFIG --variable libexec_bin xenlight`
> - with_libxl=yes
> - ], [LIBXL_FOUND=no])
...
> + old_with_libxl="$with_libxl"
> + LIBVIRT_CHECK_PKG([LIBXL], [xenlight], [4.4.0], [true])
Why 4.4.0? The comment above says pkgconfig file was introduced in 4.5
and libvirt requires libxenlight 4.2. So either this check should
require libxenlight 4.2.0 (if we are brave and want to check a possible
downstream pkgconfig file) or 4.5.0.
Jirka
More information about the libvir-list
mailing list