[Lldb-commits] [PATCH] D55013: [CMake] Streamline code signing for debugserver #2
Jonas Devlieghere via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Dec 4 10:08:32 PST 2018
JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.
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
> JDevlieghere wrote:
> > 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?
> This is the top level CMakeLists.txt for standalone debugserver :) See my previous comment about testing.
Thanks. I'm not super excited about duplicating the variables but I understand why it's needed. I think long term it'd be great if we could extract some stuff form the top-level cmake file and just make it dispatch things, but that's beyond the scope of what you're doing right now.
CHANGES SINCE LAST ACTION
More information about the lldb-commits