[libvirt] [PATCH] m4: Don't enable bash-completion by default

Michal Privoznik mprivozn at redhat.com
Thu Jan 11 21:22:16 UTC 2018


Due to the way that check logic was written we basically enabled
bash completion whenever readline was enabled. This is not right
because it made bash-completion pkg-config module required.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---

Even though this would qualify as a build breaker, I'd rather have
another pair of eyes to look at it (sorry Cyclops).

 m4/virt-bash-completion.m4 | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/m4/virt-bash-completion.m4 b/m4/virt-bash-completion.m4
index e1ef58740..c8342176f 100644
--- a/m4/virt-bash-completion.m4
+++ b/m4/virt-bash-completion.m4
@@ -30,14 +30,10 @@ AC_DEFUN([LIBVIRT_CHECK_BASH_COMPLETION], [
   AC_REQUIRE([LIBVIRT_CHECK_READLINE])
 
   if test "x$with_readline" != "xyes" ; then
-    if test "x$with_bash_completion" != "xyes" ; then
-      with_bash_completion=no
-    else
+    if test "x$with_bash_completion" = "xyes" ; then
       AC_MSG_ERROR([readline is required for bash completion support])
-    fi
-  else
-    if test "x$with_bash_completion" = "xcheck" ; then
-      with_bash_completion=yes
+    else
+      with_bash_completion=no
     fi
   fi
 
-- 
2.13.6




More information about the libvir-list mailing list