[asterisk-dev] [Code Review] Allow linking/building against an external editline library.

Kevin Fleming reviewboard at asterisk.org
Wed Jul 25 06:55:11 CDT 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1528/#review6802
-----------------------------------------------------------


I'm going to commit a heavily modified version of this patch that has been tested on Ubuntu 12.04 and Fedora 17.

- Kevin


On Oct. 18, 2011, 8:09 a.m., jcollie wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1528/
> -----------------------------------------------------------
> 
> (Updated Oct. 18, 2011, 8:09 a.m.)
> 
> 
> Review request for Asterisk Developers and Russell Bryant.
> 
> 
> Summary
> -------
> 
> The Asterisk tarball includes a very old copy of the editline library
> that manages command line history and editing.  Fedora (and several
> other distributions) highly discourage this practice because if there
> are security problems (or even just severe bugs) in the library it can
> be very difficult to track down all instances of the library.  This
> patch adds the option of building/linking against an external copy of
> the editline library.
> 
> 
> This addresses bug ASTERISK-18725.
>     https://issues.asterisk.org/jira/browse/ASTERISK-18725
> 
> 
> Diffs
> -----
> 
>   /trunk/build_tools/menuselect-deps.in 341072 
>   /trunk/configure UNKNOWN 
>   /trunk/configure.ac 341072 
>   /trunk/include/asterisk/autoconfig.h.in 341072 
>   /trunk/main/Makefile 341072 
>   /trunk/main/asterisk.c 341072 
>   /trunk/main/cli.c 341072 
>   /trunk/makeopts.in 341072 
> 
> Diff: https://reviewboard.asterisk.org/r/1528/diff
> 
> 
> Testing
> -------
> 
> This patch has been in use in the Fedora Asterisk packages for some time now.  Cleaning up and submitting for review.  Has been compiled and CLI editing and recall works as it's supposed to.
> 
> 
> Thanks,
> 
> jcollie
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120725/f265f937/attachment.htm>


More information about the asterisk-dev mailing list