[Home]

Summary:ASTERISK-06764: [patch] asterisk does not build when there is static ncurses library present on a box
Reporter:Andrey S Pankov (casper)Labels:
Date Opened:2006-04-12 13:27:16Date Closed:2006-05-08 10:25:02
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 6948-v2.diff
Description:This is reproducible and is a bug in asterisk.

This happens only when libncurses compiled --with-terminfo.
Just install static ncurses and tinfo libraries and asterisk
does not build anymore.
Comments:By: Andrey S Pankov (casper) 2006-04-15 12:34:45

Even if there is no ncurses installed at all and editline uses termcap the proposed patch works!

I'd pass -Wl,-E,--as-needed to ld in any case (this is not in the patch)...

By: Tilghman Lesher (tilghman) 2006-04-16 01:13:28

How is a lack of a stated requirement to build Asterisk considered a bug?  Isn't this really just that the sysadmin has forgotten to install the requisite package?

By: Andrey S Pankov (casper) 2006-04-16 11:02:24

Let me explain the issue again... ;)

Pre-require: ncurses are build --with-terminfo, both shared and static.

Require: Both shared and static ncurses and tinfo are installed:

/usr/lib/libncurses.a
/usr/lib/libncurses.so
/usr/lib/libncurses.so.3
/usr/lib/libncurses.so.4
/usr/lib/libncurses.so.5
/usr/lib/libncurses.so.5.4

/usr/lib/libtinfo.a
/usr/lib/libtinfo.so
/lib/libtinfo.so.5
/lib/libtinfo.so.5.4

This is the first system configuration.

The second one is when no ncurses installed at all and editline is using termcap (happend on Slackware 10.2).

The proposed solution fixes both cases (but on Linux only for now).

By: Andrey S Pankov (casper) 2006-04-24 16:56:50

I've uploaded a patch which solves the problem using autoconf...
plus some minor output beautifications.

By: Andrey S Pankov (casper) 2006-04-25 12:36:34

Russell: could you look at this? It seems it needs to be intergrated with your latest curses/ncurses stuff changes for menuselect...

By: Serge Vecher (serge-v) 2006-05-04 16:27:19

casper: which of these 3 files need to be applied? Can you please make one patch. Thanks, man.

By: Andrey S Pankov (casper) 2006-05-04 17:12:38

Updated...

By: Denis Smirnov (mithraen) 2006-05-07 14:44:06

In some distro ncurses splitted.
And we need to link with tinfo, not ncurses.
This patch fix it, I think.

By: Serge Vecher (serge-v) 2006-05-07 15:09:31

why do you add spaces in several lines like:
-   echo -n "checking for vpb_open in -lvpb..."
+   echo -n "checking for vpb_open in -lvpb... "

By: Andrey S Pankov (casper) 2006-05-07 15:20:27

vechers: just look at the configure output - we have a space everywhere
before yes or no, but not in these cases.

By: Andrey S Pankov (casper) 2006-05-07 15:21:41

And this is not a whitespace change (like formatting), this is a fix
to be consistent within the script... :)

By: Denis Smirnov (mithraen) 2006-05-08 05:25:41

I think it works fine.

By: Russell Bryant (russell) 2006-05-08 10:25:01

a modified version of this has been added to the trunk in revision 25611