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

jcollie reviewboard at asterisk.org
Tue Oct 18 08:09:29 CDT 2011


-----------------------------------------------------------
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.


Changes
-------

Fix compiling of asterisk.c when external editline lib is not available.


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 (updated)
-----

  /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/20111018/b2432ed3/attachment.htm>


More information about the asterisk-dev mailing list