[Lldb-commits] [lldb] r333777 - Add dependency on clang-headers when building LLDB.framework using CMake

[Alex Langford via lldb-commits]

Author: xiaobai
Date: Fri Jun  1 11:14:49 2018
New Revision: 333777

URL: http://llvm.org/viewvc/llvm-project?rev=333777&view=rev
Log:
Add dependency on clang-headers when building LLDB.framework using CMake

Summary:
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.

Reviewers: clayborg, labath, sas

Subscribers: mgorny, lldb-commits

Differential Revision: https://reviews.llvm.org/D47612

Modified:
    lldb/trunk/source/API/CMakeLists.txt

Modified: lldb/trunk/source/API/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/API/CMakeLists.txt?rev=333777&r1=333776&r2=333777&view=diff
==============================================================================
--- lldb/trunk/source/API/CMakeLists.txt (original)
+++ lldb/trunk/source/API/CMakeLists.txt Fri Jun  1 11:14:49 2018
@@ -187,6 +187,9 @@ if(LLDB_BUILD_FRAMEWORK)
     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