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

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

 



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@xxxxxxxxxx>
---

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

--
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