Review Request 118959: Fix linking to ncurses

[Aleix Pol Gonzalez]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118959/#review61059
-----------------------------------------------------------



CMakeLists.txt
<https://git.reviewboard.kde.org/r/118959/#comment42519>

    Remove this



CMakeLists.txt
<https://git.reviewboard.kde.org/r/118959/#comment42520>

    CURSES_FOUND, not READLINE_FOUND



CMakeLists.txt
<https://git.reviewboard.kde.org/r/118959/#comment42518>

    Change this to:
    if(READLINE_FOUND AND CURSES_FOUND)



calgebra/CMakeLists.txt
<https://git.reviewboard.kde.org/r/118959/#comment42521>

    Can we move that check to FindReadline.cmake?


- Aleix Pol Gonzalez


On June 26, 2014, 10:48 p.m., Vadim Zhukov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118959/
> -----------------------------------------------------------
> 
> (Updated June 26, 2014, 10:48 p.m.)
> 
> 
> Review request for KDE Edu.
> 
> 
> Repository: analitza
> 
> 
> Description
> -------
> 
> On some systems, libreadline is linked to Curses implementation directly; on other, it's not, requiring you to link both Curses and Readline libraries manually.
> 
> Also, OpenBSD ships with old libreadline that lacks free_history_entry().
> 
> I've added all required checks, and forced linking to NCurses library, too.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 09e216f 
>   calgebra/CMakeLists.txt 1cbb06b 
>   calgebra/main.cpp fa045ed 
> 
> Diff: https://git.reviewboard.kde.org/r/118959/diff/
> 
> 
> Testing
> -------
> 
> OpenBSD/i386-CURRENT
> 
> 
> Thanks,
> 
> Vadim Zhukov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20140626/a0ae2c29/attachment.html>