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

Paul Belanger reviewboard at asterisk.org
Mon Oct 17 10:46:02 CDT 2011


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



/trunk/main/cli.c
<https://reviewboard.asterisk.org/r/1528/#comment8692>

    This need to be:
    #include <editline/readline.h>
    
    for Debian / Ubuntu to compile
    


- Paul


On Oct. 14, 2011, 7:32 p.m., jcollie wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1528/
> -----------------------------------------------------------
> 
> (Updated Oct. 14, 2011, 7:32 p.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.ac 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/20111017/c84ffb53/attachment.htm>


More information about the asterisk-dev mailing list