[Lldb-commits] [PATCH] D12672: add a dependency on terminfo library if llvm uses it

Jeremi Piotrowski via lldb-commits lldb-commits at lists.llvm.org
Mon Sep 7 13:15:20 PDT 2015


jeremi.piotrowski updated this revision to Diff 34169.
jeremi.piotrowski added a comment.

- combine nested if conditions with `and` as suggested by zturner

  and remove condtition from `endif` as that seems to be the convention.


http://reviews.llvm.org/D12672

Files:
  cmake/LLDBDependencies.cmake

Index: cmake/LLDBDependencies.cmake
===================================================================
--- cmake/LLDBDependencies.cmake
+++ cmake/LLDBDependencies.cmake
@@ -146,6 +146,9 @@
   endif()
   if (NOT LLDB_DISABLE_CURSES)
     list(APPEND LLDB_SYSTEM_LIBS panel ncurses)
+    if(LLVM_ENABLE_TERMINFO AND HAVE_TERMINFO)
+      list(APPEND LLDB_SYSTEM_LIBS ${TERMINFO_LIBS})
+    endif()
   endif()
 endif()
 # On FreeBSD backtrace() is provided by libexecinfo, not libc.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12672.34169.patch
Type: text/x-patch
Size: 481 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150907/75cd5fe7/attachment.bin>


More information about the lldb-commits mailing list