[clang-tools-extra] r175744 - Fixing a "multiple rules generate X" warning from ninja

Edwin Vane edwin.vane at intel.com
Thu Feb 21 07:12:01 PST 2013


Author: revane
Date: Thu Feb 21 09:12:01 2013
New Revision: 175744

URL: http://llvm.org/viewvc/llvm-project?rev=175744&view=rev
Log:
Fixing a "multiple rules generate X" warning from ninja

CMake's Ninja generator was not detecting that test/lit.site.cfg.in and
test/subdir/../lit.site.cfg.in were really the same file. Ninja noticed this
and complained as both appeared as targets (for the missing file rule). Now
canonicalizing the path to ensure the paths presented to CMake are identical
and the duplication is now fixed.


Modified:
    clang-tools-extra/trunk/test/cpp11-migrate/CMakeLists.txt

Modified: clang-tools-extra/trunk/test/cpp11-migrate/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/cpp11-migrate/CMakeLists.txt?rev=175744&r1=175743&r2=175744&view=diff
==============================================================================
--- clang-tools-extra/trunk/test/cpp11-migrate/CMakeLists.txt (original)
+++ clang-tools-extra/trunk/test/cpp11-migrate/CMakeLists.txt Thu Feb 21 09:12:01 2013
@@ -53,7 +53,8 @@ add_custom_target(cpp11-migrate-autogen
 set(TEST_SOURCE_ROOT ${CMAKE_CURRENT_BINARY_DIR}/generated_tests)
 set(TEST_EXEC_ROOT ${CMAKE_CURRENT_BINARY_DIR})
 set(TESTSUITE_NAME "cpp11-migrate Auto-Generated Tests")
+get_filename_component(INPUT_LIT_CFG ${CMAKE_CURRENT_SOURCE_DIR}/../lit.site.cfg.in REALPATH)
 configure_lit_site_cfg(
-  ${CMAKE_CURRENT_SOURCE_DIR}/../lit.site.cfg.in
+  ${INPUT_LIT_CFG}
   ${CMAKE_CURRENT_BINARY_DIR}/generated_tests/lit.site.cfg
   )





More information about the cfe-commits mailing list