Add back lldb-framework-headers target
authorAlex Langford <apl@fb.com>
Thu, 26 Jul 2018 21:55:14 +0000 (21:55 +0000)
committerAlex Langford <apl@fb.com>
Thu, 26 Jul 2018 21:55:14 +0000 (21:55 +0000)
In r338058 we removed the target `lldb-framework-headers`, which mean
lldb-framework no longer depended on `framework_headers`, so they never
actually got generated. This is a partial revert of r338058: I added
back the lldb-framework-headers target, but the framework-header-fix.sh
script still runs on the copied headers.

llvm-svn: 338074

lldb/cmake/modules/LLDBFramework.cmake

index 4ef0d3a..abacee8 100644 (file)
@@ -13,6 +13,8 @@ foreach(header
   list(APPEND framework_headers ${CMAKE_CURRENT_BINARY_DIR}/FrameworkHeaders/${basename})
 endforeach()
 
+add_custom_target(lldb-framework-headers DEPENDS ${framework_headers})
+
 add_custom_command(TARGET lldb-framework POST_BUILD
   COMMAND ${CMAKE_COMMAND} -E copy_directory ${CMAKE_CURRENT_BINARY_DIR}/FrameworkHeaders $<TARGET_FILE_DIR:liblldb>/Headers
   COMMAND ${LLDB_SOURCE_DIR}/scripts/framework-header-fix.sh $<TARGET_FILE_DIR:liblldb>/Headers ${LLDB_VERSION}
@@ -38,4 +40,5 @@ set_target_properties(liblldb PROPERTIES
   PUBLIC_HEADER "${framework_headers}")
 
 add_dependencies(lldb-framework
+  lldb-framework-headers
   lldb-suite)