build-sys: prefer ncurses-config rather than pkg-config

If you have installed:

 ii  libncurses5:i386      5.9+20140913-1+b1  i386   shared libraries for terminal handling
 ii  libncurses5-dev:i386  5.9+20140913-1+b1  i386   developer's libraries for ncurses
 ii  libncursesw5:i386     5.9+20140913-1+b1  i386   shared libraries for terminal handling (wide character support)

then pkg-config blindly follows ncursesw although there are not header
files for this library. It seems better to use pkg-config as fallback
solution only.

Signed-off-by: Karel Zak <kzak@redhat.com>

diff --git a/m4/ul.m4 b/m4/ul.m4
index db04c12691d74e27034a0b80880d0a781e600f9b..cfd2e68fc5c154ffa91ebaf05c6be023b579757c 100644 (file)
--- a/m4/ul.m4
+++ b/m4/ul.m4
@@ -455,14 +455,6 @@ AC_DEFUN([UL_NCURSES_CHECK], [
   m4_define([suffix], $1)
   m4_define([SUFFIX], m4_toupper($1))
 
-  # pkg-config (not supported by ncurses upstream by default)
-  #
-  PKG_CHECK_MODULES(SUFFIX, [$1], [
-    have_[]suffix=yes
-    NCURSES_LIBS=${SUFFIX[]_LIBS}
-    NCURSES_CFLAGS=${SUFFIX[]_CFLAGS}
-  ],[have_[]suffix=no])
-
   # ncurses6-config
   #
   AS_IF([test "x$have_[]suffix" = xno], [
@@ -489,12 +481,20 @@ AC_DEFUN([UL_NCURSES_CHECK], [
     fi
   ])
 
+  # pkg-config (not supported by ncurses upstream by default)
+  #
+  AS_IF([test "x$have_[]suffix" = xno], [
+    PKG_CHECK_MODULES(SUFFIX, [$1], [
+      have_[]suffix=yes
+      NCURSES_LIBS=${SUFFIX[]_LIBS}
+      NCURSES_CFLAGS=${SUFFIX[]_CFLAGS}
+    ],[have_[]suffix=no])
+  ])
+
   # classic autoconf way
   #
   AS_IF([test "x$have_[]suffix" = xno], [
-    AS_IF([test "x$have_[]suffix" = xno], [
-      AC_CHECK_LIB([$1], [initscr], [have_[]suffix=yes], [have_[]suffix=no])
-      AS_IF([test "x$have_[]suffix" = xyes], [NCURSES_LIBS="-l[]suffix"])
-    ])
+    AC_CHECK_LIB([$1], [initscr], [have_[]suffix=yes], [have_[]suffix=no])
+    AS_IF([test "x$have_[]suffix" = xyes], [NCURSES_LIBS="-l[]suffix"])
   ])
 ])