Looks good <br><div class="gmail_quote">On Fri, Apr 24, 2015 at 2:07 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi zturner,<br>
<br>
Currently, linking of the unittests fails on linux because it is missing a bunch of symbols from<br>
libedit, curses, etc. This fixes the build by adding the correct dependencies.<br>
<br>
<a href="http://reviews.llvm.org/D9244" target="_blank">http://reviews.llvm.org/D9244</a><br>
<br>
Files:<br>
  unittests/CMakeLists.txt<br>
<br>
Index: unittests/CMakeLists.txt<br>
===================================================================<br>
--- unittests/CMakeLists.txt<br>
+++ unittests/CMakeLists.txt<br>
@@ -23,7 +23,7 @@<br>
   endif()<br>
<br>
   lldb_link_common_libs(${test_name} EXE)<br>
-  target_link_libraries(${test_name} ${CLANG_USED_LIBS})<br>
+  target_link_libraries(${test_name} ${CLANG_USED_LIBS} ${LLDB_SYSTEM_LIBS})<br>
   llvm_config(${test_name} ${LLVM_LINK_COMPONENTS})<br>
 endfunction()<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
</blockquote></div>