tags 646740 + patch
thanks

On 2011-10-26 19:30 +0200, Sven Joachim wrote:

> Source: malaga
> Version: 7.12-3.1
> Severity: important
> User: [email protected]
> Usertags: readline-disabled
>
> When built in a clean sid chroot without libncurses5-dev installed,
> malaga loses readline support.  From my pbuilder log:
>
> ,----
> | checking for readline in -lreadline... no
> | checking readline/readline.h usability... yes
> | checking readline/readline.h presence... yes
> | checking for readline/readline.h... yes
> `----
>
> Adding libncurses-dev to Build-Depends should avoid this.  However,
> consider patching the upstream build system which erroneously believes
> that curses is necessary for linking with readline.

Attached i a debdiff with a patch that works for me.  A cleaner solution
might be to only patch configure.in, remove configure and other
generated files in the clean target and run autoconf at build time
(adding the necessary Build-Depends).

Cheers,
       Sven

diff -u malaga-7.12/debian/changelog malaga-7.12/debian/changelog
--- malaga-7.12/debian/changelog
+++ malaga-7.12/debian/changelog
@@ -1,3 +1,10 @@
+malaga (7.12-3.2) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Do not unnecessesarily link against ncurses (Closes: #646740).
+
+ -- Sven Joachim <[email protected]>  Sat, 03 Dec 2011 12:13:18 +0100
+
 malaga (7.12-3.1) unstable; urgency=low
 
   * Non-maintainer upload.
diff -u malaga-7.12/debian/patches/series malaga-7.12/debian/patches/series
--- malaga-7.12/debian/patches/series
+++ malaga-7.12/debian/patches/series
@@ -3,0 +4 @@
+no-ncurses.diff
only in patch2:
unchanged:
--- malaga-7.12.orig/debian/patches/no-ncurses.diff
+++ malaga-7.12/debian/patches/no-ncurses.diff
@@ -0,0 +1,47 @@
+Author: Sven Joachim <[email protected]>
+Bug-Debian: http://bugs.debian.org/646740
+Forwarded: no
+Last-Update: 2011-12-03
+
+---
+ configure    |    4 ++--
+ configure.in |    4 ++--
+ 2 files changed, 4 insertions(+), 4 deletions(-)
+
+--- a/configure.in
++++ b/configure.in
+@@ -39,12 +39,12 @@
+             use_readline=$withval, use_readline=yes)
+ if test $use_readline = yes
+ then
+-  AC_CHECK_LIB(readline,readline,,use_readline=no,-lcurses)
++  AC_CHECK_LIB(readline,readline,,use_readline=no)
+   AC_CHECK_HEADER(readline/readline.h,,use_readline=no)
+ fi
+ if test $use_readline = yes 
+ then
+-  READLINE_LIBS="-lreadline -lcurses"
++  READLINE_LIBS="-lreadline"
+   READLINE_CFLAGS="-DREADLINE"
+ else
+   READLINE_LIBS=""
+--- a/configure
++++ b/configure
+@@ -18462,7 +18462,7 @@
+   echo $ECHO_N "(cached) $ECHO_C" >&6
+ else
+   ac_check_lib_save_LIBS=$LIBS
+-LIBS="-lreadline -lcurses $LIBS"
++LIBS="-lreadline $LIBS"
+ cat >conftest.$ac_ext <<_ACEOF
+ /* confdefs.h.  */
+ _ACEOF
+@@ -18663,7 +18663,7 @@
+ fi
+ if test $use_readline = yes
+ then
+-  READLINE_LIBS="-lreadline -lcurses"
++  READLINE_LIBS="-lreadline"
+   READLINE_CFLAGS="-DREADLINE"
+ else
+   READLINE_LIBS=""

Reply via email to