[Lldb-commits] [lldb] r360695 - build: use the correct variable

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Tue May 14 11:02:52 PDT 2019


Hey Saleem,

Why is this necessary after Pavel's change in r360652? Now that the
 LLDB_CAN_USE_LLDB_SERVER is defined correctly, the
LLDB_TOOL_LLDB_SERVER_BUILD option should never be set.

Thanks,
Jonas

On Tue, May 14, 2019 at 10:22 AM Saleem Abdulrasool via lldb-commits <
lldb-commits at lists.llvm.org> wrote:

> Author: compnerd
> Date: Tue May 14 10:24:45 2019
> New Revision: 360695
>
> URL: http://llvm.org/viewvc/llvm-project?rev=360695&view=rev
> Log:
> build: use the correct variable
>
> Adjust the variable that controls whether the unit tests use `lldb-server`.
> This should repair the default build on Windows.
>
> Modified:
>     lldb/trunk/unittests/tools/CMakeLists.txt
>
> Modified: lldb/trunk/unittests/tools/CMakeLists.txt
> URL:
> http://llvm.org/viewvc/llvm-project/lldb/trunk/unittests/tools/CMakeLists.txt?rev=360695&r1=360694&r2=360695&view=diff
>
> ==============================================================================
> --- lldb/trunk/unittests/tools/CMakeLists.txt (original)
> +++ lldb/trunk/unittests/tools/CMakeLists.txt Tue May 14 10:24:45 2019
> @@ -1,6 +1,6 @@
>  if(LLDB_TOOL_LLDB_MI_BUILD)
>    add_subdirectory(lldb-mi)
>  endif()
> -if(LLDB_TOOL_LLDB_SERVER_BUILD)
> +if(LLDB_CAN_USE_LLDB_SERVER)
>    add_subdirectory(lldb-server)
>  endif()
>
>
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20190514/05b8600e/attachment.html>


More information about the lldb-commits mailing list