diff options
| author | Chris Bieneman <beanz@apple.com> | 2016-10-14 17:09:55 +0000 |
|---|---|---|
| committer | Chris Bieneman <beanz@apple.com> | 2016-10-14 17:09:55 +0000 |
| commit | 9a6502f1cd3ed10dd1c830499b373710ce436816 (patch) | |
| tree | b4c520ad1f14a4624212563fbb6b627ac5e92cf7 | |
| parent | 4a2ef91143f10eabeefb3c160954c4bc52b6962d (diff) | |
| download | bcm5719-llvm-9a6502f1cd3ed10dd1c830499b373710ce436816.tar.gz bcm5719-llvm-9a6502f1cd3ed10dd1c830499b373710ce436816.zip | |
[CMake] Populate LLDB.framework's headers directory
Summary:
This patch adds support for installing public headers in LLDB.framework, and symlinking the headers into the build directory.
While writing the patch I discovered a bug in CMake that prevents applying POST_BUILD steps to framework targets (https://gitlab.kitware.com/cmake/cmake/issues/16363).
I've implemented the support using POST_BUILD steps wrapped under a CMake version check with a TODO so that we can track the fix.
Reviewers: tfiala, zturner, spyffe
Subscribers: lldb-commits, mgorny
Differential Revision: https://reviews.llvm.org/D25570
llvm-svn: 284250
| -rw-r--r-- | lldb/source/API/CMakeLists.txt | 18 |
1 files changed, 17 insertions, 1 deletions
diff --git a/lldb/source/API/CMakeLists.txt b/lldb/source/API/CMakeLists.txt index 9701dfc64f8..903384657f9 100644 --- a/lldb/source/API/CMakeLists.txt +++ b/lldb/source/API/CMakeLists.txt @@ -135,10 +135,26 @@ endif() target_link_libraries(liblldb PRIVATE ${LLDB_SYSTEM_LIBS}) if(LLDB_BUILD_FRAMEWORK) + file(GLOB public_headers ${LLDB_SOURCE_DIR}/include/lldb/API/*.h) set_target_properties(liblldb PROPERTIES OUTPUT_NAME LLDB FRAMEWORK On FRAMEWORK_VERSION ${LLDB_FRAMEWORK_VERSION} - LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/${LLDB_FRAMEWORK_INSTALL_DIR}) + LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/${LLDB_FRAMEWORK_INSTALL_DIR} + PUBLIC_HEADER "${public_headers}") + + # This works around a CMake bug where POST_BUILD steps are not applied to + # framework targets. This fix is merged into the CMake release branch and + # should be available with CMake 3.7 rc2: + # https://gitlab.kitware.com/cmake/cmake/issues/16363 + if(CMAKE_VERSION VERSION_GREATER 3.6.99) + add_custom_command(TARGET liblldb POST_BUILD + COMMAND ${CMAKE_COMMAND} -E create_symlink ${LLDB_SOURCE_DIR}/include/lldb/API $<TARGET_FILE_DIR:liblldb>/Headers) + else() + add_custom_command(OUTPUT ${CMAKE_BINARY_DIR}/${CMAKE_CFG_INTDIR}/${LLDB_FRAMEWORK_INSTALL_DIR}/LLDB.framework/Versions/${LLDB_FRAMEWORK_VERSION}/Headers + COMMAND ${CMAKE_COMMAND} -E create_symlink ${LLDB_SOURCE_DIR}/include/lldb/API $<TARGET_FILE_DIR:liblldb>/Headers) + add_custom_target(lldb_header_symlink + DEPENDS ${CMAKE_BINARY_DIR}/${CMAKE_CFG_INTDIR}/${LLDB_FRAMEWORK_INSTALL_DIR}/LLDB.framework/Versions/${LLDB_FRAMEWORK_VERSION}/Headers) + endif() endif() |
