[Lldb-commits] [PATCH] D48060: Introduce lldb-framework CMake target and centralize its logic

Alex Langford via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Jun 12 16:32:42 PDT 2018

xiaobai added inline comments.

Comment at: CMakeLists.txt:145
+  add_dependencies(lldb-suite lldb-framework)
compnerd wrote:
> Shouldn't this be `else()`?

Comment at: CMakeLists.txt:176
+      # The target to install libLLDB needs to depend on finish swig so that the
+      # framework build properly copies over the Python files.
compnerd wrote:
> "finish swig"?
Should be "finish_swig"

Comment at: cmake/modules/LLDBFramework.cmake:5
+file(GLOB root_public_headers ${LLDB_SOURCE_DIR}/include/lldb/lldb-*.h)
+file(GLOB root_private_headers ${LLDB_SOURCE_DIR}/include/lldb/lldb-private-*.h)
+list(REMOVE_ITEM root_public_headers ${root_private_headers})
compnerd wrote:
> Globs are generally frowned upon for dependency tracking issues.
I see. This code was moved from somewhere else and written by somebody else. My CMake is somewhat weak, how would you do this instead?


More information about the lldb-commits mailing list