[Lldb-commits] [PATCH] D29352: [CMake] Final dependency cleanup patch!

[Pavel Labath via Phabricator via lldb-commits]

labath added a comment.

the newlines were a red herring. I think the problem is that you forgot to use the LINK_LIBS argument (see comment).

This works on linux after applying this change (and fixing ObjectFileELF test dependencies, which you missed because they were added yesterday).



================
Comment at: unittests/CMakeLists.txt:42
 
-  lldb_link_common_libs(${test_name} EXE)
   target_link_libraries(${test_name} ${CLANG_USED_LIBS} ${LLDB_SYSTEM_LIBS})
 endfunction()
----------------
add `${ARG_LINK_LIBS}` here


https://reviews.llvm.org/D29352