[Qemu-devel] [PATCH] configure: Only enable iconv if curses is enabled

Kumar Gala posted 1 patch 6 years, 8 months ago
Test docker-clang@ubuntu passed
Test asan passed
Test checkpatch passed
Test docker-mingw@fedora passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190517211805.31918-1-kumar.gala@linaro.org
configure | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
[Qemu-devel] [PATCH] configure: Only enable iconv if curses is enabled
Posted by Kumar Gala 6 years, 8 months ago
iconv is only used with if curses is enabled, there's no need to do any
configure checking for iconv, if curses is disabled.  Also, ignore
--enable-iconv if curses is already disabled.

Signed-off-by: Kumar Gala <kumar.gala@linaro.org>
---
 configure | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/configure b/configure
index d2fc346302..a1b60fa977 100755
--- a/configure
+++ b/configure
@@ -1235,13 +1235,18 @@ for opt do
   ;;
   --disable-stack-protector) stack_protector="no"
   ;;
-  --disable-curses) curses="no"
+  --disable-curses)
+      curses="no"
+      iconv="no"
   ;;
   --enable-curses) curses="yes"
   ;;
   --disable-iconv) iconv="no"
   ;;
-  --enable-iconv) iconv="yes"
+  --enable-iconv)
+      if test "$curses" != "no" ; then
+	  iconv="yes"
+      fi
   ;;
   --disable-curl) curl="no"
   ;;
-- 
2.20.1


Re: [Qemu-devel] [PATCH] configure: Only enable iconv if curses is enabled
Posted by Gerd Hoffmann 6 years, 8 months ago
On Fri, May 17, 2019 at 04:18:05PM -0500, Kumar Gala wrote:
> iconv is only used with if curses is enabled, there's no need to do any
> configure checking for iconv, if curses is disabled.  Also, ignore
> --enable-iconv if curses is already disabled.

How about just doing this ...

--- a/configure
+++ b/configure
@@ -3551,6 +3551,7 @@ EOF
       feature_not_found "curses" "Install ncurses devel"
     fi
     curses=no
+    iconv=no  # curses is the only user
   fi
 fi

... instead?