[Lldb-commits] [PATCH] D55013: [CMake] Streamline code signing for debugserver #2

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Nov 28 16:05:55 PST 2018


JDevlieghere added inline comments.


================
Comment at: tools/debugserver/CMakeLists.txt:18
+
+  option(LLDB_USE_ENTITLEMENTS "When code signing, use entitlements if available" ON)
+  set(LLDB_CODESIGN_IDENTITY lldb_codesign CACHE STRING
----------------
sgraenitz wrote:
> JDevlieghere wrote:
> > Why do we need to define this option again for the debugserver? Why can't it use the one from LLDBConfig.cmake?
> In a debugserver standalone build we don't `include(LLDBConfig)`. It makes sense to me, because there is not much to configure when only building the debugserver.
Makes sense. Maybe we could put this in the top level CMakeLists.txt? 


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55013/new/

https://reviews.llvm.org/D55013





More information about the lldb-commits mailing list