[PATCH] build: silence configure warning if ksh is missing

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

 



* configure.ac (lv_cv_wrapper_shell): Fix logic error if candidate
shell is not available.
Reported by Matthias Bolte.
---

On IRC, Matthias reported a noisy configure run:

eblake: checking for shell that supports <> redirection... ./configure: line 31581: ksh: command not found

Worse, if a shell is missing, it doesn't change the contents of
conftest.a, so that missing shell would be selected in the loop.
This patch fixes both the noise and the wrong selection.

 configure.ac |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/configure.ac b/configure.ac
index 86a864b..7e7f295 100644
--- a/configure.ac
+++ b/configure.ac
@@ -616,7 +616,7 @@ if test "$with_qemu" = yes; then
         test $lv_cv_wrapper_shell = none &&
           AC_MSG_ERROR([could not find decent shell])
         echo a > conftest.a
-        $lv_cv_wrapper_shell -c ': 1<>conftest.a'
+        ($lv_cv_wrapper_shell -c ': 1<>conftest.a') 2>/dev/null &&
         case `cat conftest.a`.$lv_cv_wrapper_shell in
           a./*) break;; dnl /bin/sh is good enough
           a.*) dnl bash, ksh, and zsh all understand 'command', use that
-- 
1.7.2.3

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