[Lldb-commits] [PATCH] LLGS Android target support

[Nico Weber]

================
Comment at: cmake/LLDBDependencies.cmake:100
@@ +99,3 @@
+if ( CMAKE_SYSTEM_NAME MATCHES "Darwin" )
+  set(LLDB_VERS_GENERATED_FILE ${LLDB_BINARY_DIR}/source/LLDB_vers.c)
+  add_custom_command(OUTPUT ${LLDB_VERS_GENERATED_FILE}
----------------
This doesn't look correct. Now every file that includes LLDBDependencies will have a custom command that builds tools/lldb/source/LLDB_vers.c. With ninja, this causes "ninja: warning: multiple rules generate tools/lldb/source/LLDB_vers.c. builds involving this target will not be correct; continuing anyway" for example (since there's more than one CMakeLists.txt that includes LLDBDependencies.txt).

Maybe this rule could stay where it was?

http://reviews.llvm.org/D6166

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/