Author: stefan.graenitz
Date: Thu Jun 13 10:35:50 2019
New Revision: 363280

URL: http://llvm.org/viewvc/llvm-project?rev=363280&view=rev
Log:
[CMake] Fix generated Xcode-project ignoring output directory setting for 
LLDB.framework

Other generators honor the `LIBRARY_OUTPUT_DIRECTORY` target property, but 
apparently Xcode doesn't. So we call `set_output_directory()` as 
`llvm_add_library()` would do and this works.
Note that `LIBRARY_OUTPUT_DIRECTORY` is still necessary, because it's used to 
store and read the target's absolute build directory (while 
`LLDB_FRAMEWORK_BUILD_DIR` is relative!).

Modified:
    lldb/trunk/cmake/modules/LLDBFramework.cmake

Modified: lldb/trunk/cmake/modules/LLDBFramework.cmake
URL: 
http://llvm.org/viewvc/llvm-project/lldb/trunk/cmake/modules/LLDBFramework.cmake?rev=363280&r1=363279&r2=363280&view=diff
==============================================================================
--- lldb/trunk/cmake/modules/LLDBFramework.cmake (original)
+++ lldb/trunk/cmake/modules/LLDBFramework.cmake Thu Jun 13 10:35:50 2019
@@ -26,6 +26,12 @@ set_target_properties(liblldb PROPERTIES
   MACOSX_FRAMEWORK_INFO_PLIST ${LLDB_SOURCE_DIR}/resources/LLDB-Info.plist.in
 )
 
+# Defined in AddLLVM.cmake; handles edge cases for multi-config generators
+set_output_directory(liblldb
+  BINARY_DIR ${framework_target_dir}
+  LIBRARY_DIR ${framework_target_dir}
+)
+
 # Affects the layout of the framework bundle (default is macOS layout).
 if(IOS)
   set_target_properties(liblldb PROPERTIES


_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to