<div dir="ltr">Looks fine, do you have commit access?  If so feel free to commit whenever.  Otherwise I can do it for you tomorrow</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 7, 2015 at 1:15 PM Jeremi Piotrowski <<a href="mailto:jeremi.piotrowski@gmail.com">jeremi.piotrowski@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jeremi.piotrowski updated this revision to Diff 34169.<br>
jeremi.piotrowski added a comment.<br>
<br>
- combine nested if conditions with `and` as suggested by zturner<br>
<br>
  and remove condtition from `endif` as that seems to be the convention.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D12672" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12672</a><br>
<br>
Files:<br>
  cmake/LLDBDependencies.cmake<br>
<br>
Index: cmake/LLDBDependencies.cmake<br>
===================================================================<br>
--- cmake/LLDBDependencies.cmake<br>
+++ cmake/LLDBDependencies.cmake<br>
@@ -146,6 +146,9 @@<br>
   endif()<br>
   if (NOT LLDB_DISABLE_CURSES)<br>
     list(APPEND LLDB_SYSTEM_LIBS panel ncurses)<br>
+    if(LLVM_ENABLE_TERMINFO AND HAVE_TERMINFO)<br>
+      list(APPEND LLDB_SYSTEM_LIBS ${TERMINFO_LIBS})<br>
+    endif()<br>
   endif()<br>
 endif()<br>
 # On FreeBSD backtrace() is provided by libexecinfo, not libc.<br>
<br>
<br>
</blockquote></div>