[Lldb-commits] [PATCH] D63745: [CMake] Check that a certificate for lldb is present at build time.
Alex Langford via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Jun 24 23:46:15 PDT 2019
xiaobai added a comment.
In D63745#1556773 <https://reviews.llvm.org/D63745#1556773>, @JDevlieghere wrote:
> On second thought, let's check that LLDB_CODESIGN_IDENTITY equals `lldb_codesign` before doing this check.
This question isn't important but I'm kind of curious: Does it have to be called lldb_codesign? Could you have an arbitrary identity and then sign with that, assuming the cert exists, or does debugserver expect a cert with that name exactly?
Comment at: lldb/CMakeLists.txt:77
+ # of failing in the middle of the build.
+ if(CMAKE_SYSTEM_NAME MATCHES "Darwin" AND TARGET debugserver)
I think `CMAKE_SYSTEM_NAME MATCHES "Darwin"` is redundant because debugserver should only be a target if you're running on Darwin, per the logic in `tools/CMakeLists.txt`.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits