smeenai created this revision.
smeenai added a reviewer: xiaobai.
Herald added a subscriber: mgorny.
We need the install-liblldb-stripped target to also depend on the
llvm-framework target in order for the installation to be guaranteed to
behave as intended. I believe this is simply a typo.
https://reviews.llvm.org/D53917
Files:
cmake/modules/AddLLDB.cmake
Index: cmake/modules/AddLLDB.cmake
===================================================================
--- cmake/modules/AddLLDB.cmake
+++ cmake/modules/AddLLDB.cmake
@@ -79,7 +79,7 @@
# framework, so it must rely on the framework being fully built first.
if (LLDB_BUILD_FRAMEWORK AND ${name} STREQUAL "liblldb")
add_dependencies(install-${name} lldb-framework)
- add_dependencies(install-lldb-framework-stripped lldb-framework)
+ add_dependencies(install-${name}-stripped lldb-framework)
endif()
endif()
endif()
Index: cmake/modules/AddLLDB.cmake
===================================================================
--- cmake/modules/AddLLDB.cmake
+++ cmake/modules/AddLLDB.cmake
@@ -79,7 +79,7 @@
# framework, so it must rely on the framework being fully built first.
if (LLDB_BUILD_FRAMEWORK AND ${name} STREQUAL "liblldb")
add_dependencies(install-${name} lldb-framework)
- add_dependencies(install-lldb-framework-stripped lldb-framework)
+ add_dependencies(install-${name}-stripped lldb-framework)
endif()
endif()
endif()
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
