Work around a really frustrating apparant CMake bug.
authorChandler Carruth <chandlerc@gmail.com>
Mon, 2 Jul 2012 21:14:06 +0000 (21:14 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Mon, 2 Jul 2012 21:14:06 +0000 (21:14 +0000)
No functionality changed here, except that the CMake installed by
default on Ubuntu Lucid should actually work with the makefile
generators now.

Thanks to Matt for the report and head-desking required to figure out
why it was failing.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@159588 91177308-0d34-0410-b5e6-96231b3b80d8

test/CMakeLists.txt

index 651d0243f14a9b9ca40aecb2360305f6544f3fbe..ea7b7ab2d4b04f15a199e67bcfb7276804a0a2a1 100644 (file)
@@ -11,14 +11,16 @@ add_lit_testsuite(check-llvm "Running the LLVM regression tests"
   ${CMAKE_CURRENT_BINARY_DIR}
   PARAMS llvm_site_config=${CMAKE_CURRENT_BINARY_DIR}/lit.site.cfg
          llvm_unit_site_config=${CMAKE_CURRENT_BINARY_DIR}/Unit/lit.site.cfg
-  DEPENDS UnitTests
-          BugpointPasses LLVMHello
+  DEPENDS BugpointPasses LLVMHello
           llc lli llvm-ar llvm-as llvm-dis llvm-extract llvm-dwarfdump
           llvm-link llvm-mc llvm-nm llvm-objdump llvm-readobj
           macho-dump opt
           FileCheck count not
   )
 set_target_properties(check-llvm PROPERTIES FOLDER "Tests")
+# Note, this is kept here rather than in the DEPENDS above because of bugs in
+# some CMake versions that mishandle the dependency otherwise.
+add_dependencies(check-llvm UnitTests)
 
 # Setup a legacy alias for 'check-llvm'. This will likely change to be an
 # alias for 'check-all' at some point in the future.