[Lldb-commits] [PATCH] D47612: Add dependency on clang-headers when building LLDB.framework using CMake

[Alex Langford via Phabricator via lldb-commits]

xiaobai created this revision.
xiaobai added reviewers: clayborg, labath, sas.
Herald added a subscriber: mgorny.

The LLDB.framework generated when building with CMake + Ninja/Make is
completely missing the clang headers. Although the code to copy them exists, we
don't even generate them unless we're building LLDB standalone.


https://reviews.llvm.org/D47612

Files:
  source/API/CMakeLists.txt


Index: source/API/CMakeLists.txt
===================================================================
--- source/API/CMakeLists.txt
+++ source/API/CMakeLists.txt
@@ -187,6 +187,9 @@
     PUBLIC_HEADER "${framework_headers}")
 
   if(NOT IOS)
+    if (NOT LLDB_BUILT_STANDALONE)
+      add_dependencies(liblldb clang-headers)
+    endif()
     add_custom_command(TARGET liblldb POST_BUILD
       COMMAND ${CMAKE_COMMAND} -E copy_directory ${CMAKE_CURRENT_BINARY_DIR}/FrameworkHeaders $<TARGET_FILE_DIR:liblldb>/Headers
       COMMAND ${CMAKE_COMMAND} -E create_symlink Versions/Current/Headers ${CMAKE_BINARY_DIR}/${LLDB_FRAMEWORK_INSTALL_DIR}/LLDB.framework/Headers


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D47612.149380.patch
Type: text/x-patch
Size: 665 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20180531/318d0179/attachment.bin>