Oops just realized I responded directly to Stu instead of the devel@pidgin.im<br><br><b><i>Bill Fassler <bill.fassler@yahoo.com></i></b> wrote:<blockquote class="replbq" style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;"> Date: Tue, 26 Feb 2008 12:50:28 -0800 (PST)<br>From: Bill Fassler <bill.fassler@yahoo.com><br>Subject: Re: request modification to configure.ac<br>To: Stu Tomlinson <stu@nosnilmot.com><br><br> I have to swap the order of the lines and add the dependency. Hopefully the patch will make it easier to see what I mean.  I can't imagine this adversely affecting the mainstream desktop users of pidgin or finch, but I haven't verified this.<br><br>Bill<br><br><b><i>Stu Tomlinson <stu@nosnilmot.com></i></b> wrote:<blockquote class="replbq" style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;"> On Tue, 2008-02-26 at 12:07 -0800, Bill Fassler wrote:<br>> Even with the
 newest pidgin release I continue have to manually modify<br>> your configure.ac file in order to ensure that -lpanel preceeds<br>> -lncurses during linking.  You folks helped me identify this issue the<br>> first time I encountered it so I am wondering if this change affects<br>> the mainstream users?  If not would you considering making this change<br>> in future releases?<br>> <br>>     AC_CHECK_LIB(panelw, update_panels,  [GNT_LIBS="$GNT_LIBS<br>> -lpanelw"], [enable_consoleui=no], [-lncursesw])<br>>     AC_CHECK_LIB(ncursesw, initscr, [GNT_LIBS="$GNT_LIBS -lncursesw"],<br>> [enable_consoleui=no])<br>> <br>> and<br>> <br>>                 AC_CHECK_LIB(panel, update_panels,<br>> [GNT_LIBS="$GNT_LIBS -lpanel"], [enable_consoleui=no], [-lncurses])<br>>         AC_CHECK_LIB(ncurses, initscr, [GNT_LIBS="$GNT_LIBS<br>> -lncurses"], [enable_consoleui=no])<br><br>Is it just the order of the checks you need changing? or do you
 just<br>need the 4th argument to AC_CHECK_LIB for panel{,w}? It would be easier<br>to see exactly what you want changing if you can supply a patch.<br><br>Regards,<br><br><br>Stu.<br><br></blockquote><br><div>        </div><hr size="1">Looking for last minute shopping deals? <a href="http://us.rd.yahoo.com/evt=51734/*http://tools.search.yahoo.com/newsearch/category.php?category=shopping">  Find them fast with Yahoo! Search.</a>--- configure.ac~ 2007-12-07 09:36:58.000000000 -0500<br>+++ configure.ac 2008-02-26 13:25:00.000000000 -0500<br>@@ -566,8 +566,8 @@<br>   [compile finch against the ncurses includes in DIR])],<br>   [ac_ncurses_includes="$withval"], [ac_ncurses_includes=""])<br> if test "x$enable_consoleui" = "xyes"; then<br>- AC_CHECK_LIB(ncursesw, initscr, [GNT_LIBS="-lncursesw"], [enable_consoleui=no])<br>- AC_CHECK_LIB(panelw, update_panels, [GNT_LIBS="$GNT_LIBS -lpanelw"], [enable_consoleui=no])<br>+ AC_CHECK_LIB(panelw, update_panels, [GNT_LIBS="$GNT_LIBS
 -lpanelw"], [enable_consoleui=no], [-lncursesw])<br>+ AC_CHECK_LIB(ncursesw, initscr, [GNT_LIBS="$GNT_LIBS -lncursesw"], [enable_consoleui=no])<br> <br>  if test "x$enable_consoleui" = "xyes"; then<br>   dnl # Some distros put the headers in ncursesw/, some don't<br>@@ -609,8 +609,8 @@<br>  else<br>   # ncursesw was not found. Look for plain old ncurses<br>   enable_consoleui=yes<br>-  AC_CHECK_LIB(ncurses, initscr, [GNT_LIBS="-lncurses"], [enable_consoleui=no])<br>-  AC_CHECK_LIB(panel, update_panels, [GNT_LIBS="$GNT_LIBS -lpanel"], [enable_consoleui=no])<br>+                AC_CHECK_LIB(panel, update_panels, [GNT_LIBS="$GNT_LIBS -lpanel"], [enable_consoleui=no], [-lncurses])<br>+  AC_CHECK_LIB(ncurses, initscr, [GNT_LIBS="$GNT_LIBS -lncurses"], [enable_consoleui=no])<br>   AC_DEFINE(NO_WIDECHAR, 1, [Define to 1 if you don't have wide-character support.])<br>   if test x"$ac_ncurses_includes" != "x"; then<br>   
 GNT_CFLAGS="-I$ac_ncurses_includes"<br></blockquote><br><p>
      <hr size=1>Looking for last minute shopping deals? <a href="http://us.rd.yahoo.com/evt=51734/*http://tools.search.yahoo.com/newsearch/category.php?category=shopping"> 
Find them fast with Yahoo! Search.</a>