[libvirt] [PATCH 1/2] virt-yajl.m4: Drop useless check for qemu

Michal Privoznik mprivozn at redhat.com
Wed Mar 13 15:50:37 UTC 2019


Firstly, this code is lacking AC_REQUIRE() therefore it may
happen that this code is executed before
LIBVIRT_DRIVER_CHECK_QEMU is evaluated. Secondly, the code tries
to detect installed version of qemu to learn if it uses HMP or
QMP and enable YAJL based on that. Well, we support only QMP and
also minimal required version of qemu is 1.5.0 so the check would
have enabled yajl anyway.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 m4/virt-yajl.m4 | 20 --------------------
 1 file changed, 20 deletions(-)

diff --git a/m4/virt-yajl.m4 b/m4/virt-yajl.m4
index c4ea0102a3..7e905c2f77 100644
--- a/m4/virt-yajl.m4
+++ b/m4/virt-yajl.m4
@@ -23,26 +23,6 @@ AC_DEFUN([LIBVIRT_ARG_YAJL],[
 
 AC_DEFUN([LIBVIRT_CHECK_YAJL],[
   dnl YAJL JSON library http://lloyd.github.com/yajl/
-  if test "$with_qemu:$with_yajl" = yes:check; then
-    dnl Some versions of qemu require the use of yajl; try to detect them
-    dnl here, although we do not require qemu to exist in order to compile.
-    dnl This check mirrors src/qemu/qemu_capabilities.c
-    AC_PATH_PROGS([QEMU], [qemu-kvm qemu kvm qemu-system-x86_64],
-                  [], [$PATH:/usr/bin:/usr/libexec])
-    if test -x "$QEMU"; then
-      if $QEMU -help 2>/dev/null | grep -q libvirt; then
-        with_yajl=yes
-      else
-        [qemu_version_sed='s/.*ersion \([0-9.,]*\).*/\1/']
-        qemu_version=`$QEMU -version | sed "$qemu_version_sed"`
-        case $qemu_version in
-          [[1-9]].* | 0.15.* ) with_yajl=yes ;;
-          0.* | '' ) ;;
-          *) AC_MSG_ERROR([Unexpected qemu version string]) ;;
-        esac
-      fi
-    fi
-  fi
 
   LIBVIRT_CHECK_LIB_ALT([YAJL], [yajl],
                         [yajl_parse_complete], [yajl/yajl_common.h],
-- 
2.19.2




More information about the libvir-list mailing list