[Lldb-commits] [PATCH] Add missing libraries to unittest link

Zachary Turner zturner at google.com
Fri Apr 24 10:04:38 PDT 2015


Looks good
On Fri, Apr 24, 2015 at 2:07 AM Pavel Labath <labath at google.com> wrote:

> Hi zturner,
>
> Currently, linking of the unittests fails on linux because it is missing a
> bunch of symbols from
> libedit, curses, etc. This fixes the build by adding the correct
> dependencies.
>
> http://reviews.llvm.org/D9244
>
> Files:
>   unittests/CMakeLists.txt
>
> Index: unittests/CMakeLists.txt
> ===================================================================
> --- unittests/CMakeLists.txt
> +++ unittests/CMakeLists.txt
> @@ -23,7 +23,7 @@
>    endif()
>
>    lldb_link_common_libs(${test_name} EXE)
> -  target_link_libraries(${test_name} ${CLANG_USED_LIBS})
> +  target_link_libraries(${test_name} ${CLANG_USED_LIBS}
> ${LLDB_SYSTEM_LIBS})
>    llvm_config(${test_name} ${LLVM_LINK_COMPONENTS})
>  endfunction()
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150424/00bc7b6e/attachment.html>


More information about the lldb-commits mailing list