[Cmake-commits] CMake branch, next, updated. v2.8.10.2-2275-g7acb879

Brad King brad.king at kitware.com
Fri Feb 22 08:24:52 EST 2013


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "CMake".

The branch, next has been updated
       via  7acb879850326a245aba03b63dd5849b3cd8ef6a (commit)
       via  d7f4468ca2acd86bf76b23df2868d0b5d4fc43a8 (commit)
       via  3b3f132ce5301be791b654b11fdb31635715e39f (commit)
       via  f0d1fe76db320edb30ea2340fa3a35d91e0d8e26 (commit)
       via  3bb57bab7b1490aad64f3cb0750d71d2dd445447 (commit)
      from  0256c0ae92e71295dedb0317a320e79d0a33a601 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=7acb879850326a245aba03b63dd5849b3cd8ef6a
commit 7acb879850326a245aba03b63dd5849b3cd8ef6a
Merge: 0256c0a d7f4468
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Fri Feb 22 08:24:50 2013 -0500
Commit:     CMake Topic Stage <kwrobot at kitware.com>
CommitDate: Fri Feb 22 08:24:50 2013 -0500

    Merge topic 'ExternalData-spaces' into next
    
    d7f4468 Tests: Generalize decision for 'make' tool supporting spaces
    3b3f132 Tests: Replace exec_program with execute_process
    f0d1fe7 ExternalData: Test content link with a space in its name
    3bb57ba CMake Nightly Date Stamp


http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=d7f4468ca2acd86bf76b23df2868d0b5d4fc43a8
commit d7f4468ca2acd86bf76b23df2868d0b5d4fc43a8
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Fri Feb 22 08:14:18 2013 -0500
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Fri Feb 22 08:14:18 2013 -0500

    Tests: Generalize decision for 'make' tool supporting spaces
    
    Some make tools do not support spaces inside the source tree paths.
    Decide early whether to add tests that depend on support for spaces so
    we can use the result in a decision for any test.

diff --git a/Tests/CMakeLists.txt b/Tests/CMakeLists.txt
index 9231605..d74fffd 100644
--- a/Tests/CMakeLists.txt
+++ b/Tests/CMakeLists.txt
@@ -50,6 +50,26 @@ if(BUILD_TESTING)
     set(TEST_CompileCommandOutput 1)
   endif()
 
+  set(MAKE_IS_GNU )
+  if(${CMAKE_TEST_MAKEPROGRAM} MATCHES make)
+    execute_process(COMMAND ${CMAKE_TEST_MAKEPROGRAM} no_such_target --version
+      RESULT_VARIABLE res OUTPUT_VARIABLE out ERROR_VARIABLE out)
+    if("${res}" STREQUAL "0")
+      if("${out}" MATCHES "GNU")
+        set(MAKE_IS_GNU 1)
+      endif()
+    endif()
+  endif()
+
+  # some old versions of make simply cannot handle spaces in paths
+  if (MAKE_IS_GNU OR
+      "${CMAKE_TEST_MAKEPROGRAM}" MATCHES "nmake|gmake|wmake" OR
+      "${CMAKE_TEST_GENERATOR}" MATCHES "Visual Studio|XCode|Borland")
+    set(MAKE_SUPPORTS_SPACES 1)
+  else()
+    set(MAKE_SUPPORTS_SPACES 0)
+  endif()
+
   set(build_generator_args
     --build-generator ${CMAKE_TEST_GENERATOR}
     --build-makeprogram ${CMAKE_TEST_MAKEPROGRAM}
@@ -1238,22 +1258,8 @@ ${CMake_BINARY_DIR}/bin/cmake -DVERSION=master -P ${CMake_SOURCE_DIR}/Utilities/
       )
     list(APPEND TEST_BUILD_DIRS "${CMake_BINARY_DIR}/Tests/kwsys")
   endif()
-  set(MAKE_IS_GNU )
-  if(${CMAKE_TEST_MAKEPROGRAM} MATCHES make)
-    execute_process(COMMAND ${CMAKE_TEST_MAKEPROGRAM} no_such_target --version
-      RESULT_VARIABLE res OUTPUT_VARIABLE out ERROR_VARIABLE out)
-    if("${res}" STREQUAL "0")
-      if("${out}" MATCHES "GNU")
-        set(MAKE_IS_GNU 1)
-      endif()
-    endif()
-  endif()
 
-  # only add this test on platforms that support it
-  # some old versions of make simply cannot handle spaces in paths
-  if (MAKE_IS_GNU OR
-      "${CMAKE_TEST_MAKEPROGRAM}" MATCHES "nmake|gmake|wmake" OR
-      "${CMAKE_TEST_GENERATOR}" MATCHES "Visual Studio|XCode|Borland")
+  if(MAKE_SUPPORTS_SPACES)
     add_test(SubDirSpaces ${CMAKE_CTEST_COMMAND}
       --build-and-test
       "${CMake_SOURCE_DIR}/Tests/SubDirSpaces"
@@ -2396,7 +2402,7 @@ ${CMake_BINARY_DIR}/bin/cmake -DVERSION=master -P ${CMake_SOURCE_DIR}/Utilities/
 
   if(TEST_CompileCommandOutput)
     set(CompileCommandOutput_EXTRA_OPTIONS
-      --build-options -DMAKE_SUPPORTS_SPACES=${MAKE_IS_GNU})
+      --build-options -DMAKE_SUPPORTS_SPACES=${MAKE_SUPPORTS_SPACES})
     ADD_TEST_MACRO(CompileCommandOutput
       "${CMake_BINARY_DIR}/Tests/CMakeLib/runcompilecommands")
   endif()

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=3b3f132ce5301be791b654b11fdb31635715e39f
commit 3b3f132ce5301be791b654b11fdb31635715e39f
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Fri Feb 22 08:08:12 2013 -0500
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Fri Feb 22 08:09:07 2013 -0500

    Tests: Replace exec_program with execute_process
    
    Drop use of the old exec_program command from CMake's own CMakeLists.txt
    files.

diff --git a/Tests/CMakeLists.txt b/Tests/CMakeLists.txt
index f8e4afd..9231605 100644
--- a/Tests/CMakeLists.txt
+++ b/Tests/CMakeLists.txt
@@ -384,7 +384,8 @@ if(BUILD_TESTING)
   # mainly it tests that cmake doesn't crash when generating these project files.
   if(${CMAKE_TEST_GENERATOR} MATCHES "Unix Makefiles"  OR  ${CMAKE_TEST_GENERATOR} MATCHES "KDevelop")
     # check which generators we have
-    exec_program(${CMAKE_CMAKE_COMMAND} ARGS --help OUTPUT_VARIABLE cmakeOutput )
+    execute_process(COMMAND ${CMAKE_CMAKE_COMMAND} --help
+      OUTPUT_VARIABLE cmakeOutput ERROR_VARIABLE cmakeOutput)
     # check for the Eclipse generator
     if ("${cmakeOutput}" MATCHES Eclipse)
       add_test(Simple_EclipseGenerator ${CMAKE_CTEST_COMMAND}
@@ -1239,11 +1240,9 @@ ${CMake_BINARY_DIR}/bin/cmake -DVERSION=master -P ${CMake_SOURCE_DIR}/Utilities/
   endif()
   set(MAKE_IS_GNU )
   if(${CMAKE_TEST_MAKEPROGRAM} MATCHES make)
-    exec_program(
-      ${CMAKE_TEST_MAKEPROGRAM} ARGS no_such_target --version
-      RETURN_VALUE res OUTPUT_VARIABLE out
-      )
-    if("${res}" EQUAL 0)
+    execute_process(COMMAND ${CMAKE_TEST_MAKEPROGRAM} no_such_target --version
+      RESULT_VARIABLE res OUTPUT_VARIABLE out ERROR_VARIABLE out)
+    if("${res}" STREQUAL "0")
       if("${out}" MATCHES "GNU")
         set(MAKE_IS_GNU 1)
       endif()

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=f0d1fe76db320edb30ea2340fa3a35d91e0d8e26
commit f0d1fe76db320edb30ea2340fa3a35d91e0d8e26
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Thu Feb 21 09:59:29 2013 -0500
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Fri Feb 22 08:04:00 2013 -0500

    ExternalData: Test content link with a space in its name
    
    Extend the Module.ExternalData test to cover a DATA{} reference whose
    name contains a space.

diff --git a/Tests/Module/ExternalData/CMakeLists.txt b/Tests/Module/ExternalData/CMakeLists.txt
index 607e9b9..35a759d 100644
--- a/Tests/Module/ExternalData/CMakeLists.txt
+++ b/Tests/Module/ExternalData/CMakeLists.txt
@@ -18,6 +18,7 @@ ExternalData_Add_Test(Data1
   NAME Data1Check
   COMMAND ${CMAKE_COMMAND}
     -D Data=DATA{Data.dat}
+    -D "DataSpace=DATA{Data Space.dat}"
     -D SeriesA=DATA{SeriesA.dat,:}
     -D SeriesB=DATA{SeriesB.dat,:}
     -D SeriesC=DATA{SeriesC.dat,:}
diff --git a/Tests/Module/ExternalData/Data Space.dat.md5 b/Tests/Module/ExternalData/Data Space.dat.md5
new file mode 100644
index 0000000..70e39bd
--- /dev/null
+++ b/Tests/Module/ExternalData/Data Space.dat.md5	
@@ -0,0 +1 @@
+8c018830e3efa5caf3c7415028335a57
diff --git a/Tests/Module/ExternalData/Data1Check.cmake b/Tests/Module/ExternalData/Data1Check.cmake
index b99d7ae..54a4156 100644
--- a/Tests/Module/ExternalData/Data1Check.cmake
+++ b/Tests/Module/ExternalData/Data1Check.cmake
@@ -2,6 +2,10 @@ file(STRINGS "${Data}" lines LIMIT_INPUT 1024)
 if(NOT "x${lines}" STREQUAL "xInput file already transformed.")
   message(SEND_ERROR "Input file:\n  ${Data}\ndoes not have expected content, but [[${lines}]]")
 endif()
+file(STRINGS "${DataSpace}" lines LIMIT_INPUT 1024)
+if(NOT "x${lines}" STREQUAL "xInput file already transformed.")
+  message(SEND_ERROR "Input file:\n  ${DataSpace}\ndoes not have expected content, but [[${lines}]]")
+endif()
 set(SeriesAn1 "1\\.dat")
 set(SeriesBn1 "_1\\.dat")
 set(SeriesCn1 "\\.1\\.dat")

-----------------------------------------------------------------------

Summary of changes:
 Source/CMakeVersion.cmake |    2 +-
 Tests/CMakeLists.txt      |   43 ++++++++++++++++++++++++-------------------
 2 files changed, 25 insertions(+), 20 deletions(-)


hooks/post-receive
-- 
CMake


More information about the Cmake-commits mailing list