[Lldb-commits] [PATCH] D36358: [lldb] [cmake] Add explicit linkage from Core to curses

mgorny created this revision. The Core library calls functions provided by the curses library. Add an appropriate explicit LINK_LIBS to ${CURSES_LIBRARIES} to propagate the dependency correctly within the build system. It seems that so far the linkage was handled by some kind of implicit magic LL

[Lldb-commits] [PATCH] D36358: [lldb] [cmake] Add explicit linkage from Core to curses

mgorny updated this revision to Diff 109879. mgorny added a comment. Forgive my poor eyesight, obviously this could go into existing LINK_LIBS ;-). https://reviews.llvm.org/D36358 Files: source/Core/CMakeLists.txt Index: source/Core/CMakeLists.txt ===

Re: [Lldb-commits] [PATCH] D36358: [lldb] [cmake] Add explicit linkage from Core to curses

Does this evaluate to nothing if curses is not present? On Sat, Aug 5, 2017 at 9:15 AM Michał Górny via Phabricator < revi...@reviews.llvm.org> wrote: > mgorny updated this revision to Diff 109879. > mgorny added a comment. > > Forgive my poor eyesight, obviously this could go into existing LINK_L

Re: [Lldb-commits] [PATCH] D36358: [lldb] [cmake] Add explicit linkage from Core to curses

Dnia 5 sierpnia 2017 19:33:57 CEST, Zachary Turner napisał(a): >Does this evaluate to nothing if curses is not present? I will test to confirm in a minute but it should since the find_package call responsible for defining it is conditional to the disable var. However, it may evaluate to a non

[Lldb-commits] [PATCH] D36358: [lldb] [cmake] Add explicit linkage from Core to curses

mgorny updated this revision to Diff 109897. mgorny edited the summary of this revision. mgorny added a comment. Updated to explicitly respect LLDB_DISABLE_CURSES and the LLVM HAVE_TERMINFO logic. https://reviews.llvm.org/D36358 Files: source/Core/CMakeLists.txt Index: source/Core/CMakeLis