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

[Saleem Abdulrasool via Phabricator via lldb-commits]

compnerd added inline comments.


================
Comment at: CMakeLists.txt:145
+  add_dependencies(lldb-suite lldb-framework)
+elseif()
+  if (LLDB_CAN_USE_LLDB_SERVER)
----------------
Shouldn't this be `else()`?


================
Comment at: CMakeLists.txt:176
+    if (LLDB_BUILD_FRAMEWORK)
+      # The target to install libLLDB needs to depend on finish swig so that the
+      # framework build properly copies over the Python files.
----------------
"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})
----------------
Globs are generally frowned upon for dependency tracking issues.


https://reviews.llvm.org/D48060