[Cmake-commits] CMake branch, next, updated. v3.3.2-3430-gadce44b

Brad King brad.king at kitware.com
Thu Oct 1 09:50:01 EDT 2015


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  adce44b9f9b85b5cf7ccaa5b25d1088925845180 (commit)
       via  1b13af46accba6ed73bf2975221def388cd5e90d (commit)
       via  981068b79f3703a095f5aa254da6c7d3f997a265 (commit)
       via  e8954d0b3b6f7b29a00ec0238fc91d84021f0705 (commit)
      from  bf7a609459b8dcbfb59b4da57c7cbe1787a56c6a (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 -----------------------------------------------------------------
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=adce44b9f9b85b5cf7ccaa5b25d1088925845180
commit adce44b9f9b85b5cf7ccaa5b25d1088925845180
Merge: bf7a609 1b13af4
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Thu Oct 1 09:50:01 2015 -0400
Commit:     CMake Topic Stage <kwrobot at kitware.com>
CommitDate: Thu Oct 1 09:50:01 2015 -0400

    Merge topic 'cpack-tests-fix' into next
    
    1b13af46 Tests: Fix RunCMake.CPack_* tests to use proper CMake generator
    981068b7 Tests: Avoid OS X 10.5 limitation warning in RunCMake.CPack* tests
    e8954d0b Tests: Add trailing newlines to generated sources in RunCMake.CPack_* tests


https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=1b13af46accba6ed73bf2975221def388cd5e90d
commit 1b13af46accba6ed73bf2975221def388cd5e90d
Author:     Domen Vrankar <domen.vrankar at gmail.com>
AuthorDate: Wed Sep 30 22:13:12 2015 +0200
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Thu Oct 1 09:48:04 2015 -0400

    Tests: Fix RunCMake.CPack_* tests to use proper CMake generator
    
    Use the run_cmake() function to generate the test build tree with
    the proper CMake generator and also to verify that it succeeds.
    Drop our PreTestError helper as it is no longer needed.

diff --git a/Tests/RunCMake/CPack/CPackTestHelpers.cmake b/Tests/RunCMake/CPack/CPackTestHelpers.cmake
index 7ea2a24..aef1086 100644
--- a/Tests/RunCMake/CPack/CPackTestHelpers.cmake
+++ b/Tests/RunCMake/CPack/CPackTestHelpers.cmake
@@ -10,47 +10,12 @@ function(run_cpack_test TEST_NAME types build)
     file(MAKE_DIRECTORY "${RunCMake_TEST_BINARY_DIR}")
 
     # execute cmake
-    execute_process(
-      COMMAND "${CMAKE_COMMAND}" -DRunCMake_TEST=${TEST_NAME}
-          -DGENERATOR_TYPE=${TEST_TYPE} "${RunCMake_SOURCE_DIR}"
-      WORKING_DIRECTORY "${RunCMake_TEST_BINARY_DIR}"
-      RESULT_VARIABLE res
-      OUTPUT_FILE "${RunCMake_TEST_BINARY_DIR}/test_output.txt"
-      ERROR_FILE "${RunCMake_TEST_BINARY_DIR}/test_error.txt"
-      )
-
-    if(res)
-      run_cmake_command(
-        ${TEST_TYPE}/${TEST_NAME}
-        "${CMAKE_COMMAND}"
-          -DRunCMake_TEST_STEP=configure
-          -Dreturn_code=${res}
-          "-Dbin_dir=${RunCMake_TEST_BINARY_DIR}"
-          -P "${RunCMake_SOURCE_DIR}/PreTestError.cmake"
-        )
-      return()
-    endif()
+    set(RunCMake_TEST_OPTIONS "-DGENERATOR_TYPE=${TEST_TYPE}")
+    run_cmake(${TEST_NAME})
 
     # execute optional build step
     if(build)
-      execute_process(
-        COMMAND "${CMAKE_COMMAND}" --build "${RunCMake_TEST_BINARY_DIR}"
-        RESULT_VARIABLE res
-        OUTPUT_FILE "${RunCMake_TEST_BINARY_DIR}/test_output.txt"
-        ERROR_FILE "${RunCMake_TEST_BINARY_DIR}/test_error.txt"
-        )
-    endif()
-
-    if(res)
-      run_cmake_command(
-        ${TEST_TYPE}/${TEST_NAME}
-        "${CMAKE_COMMAND}"
-          -DRunCMake_TEST_STEP=build
-          -Dreturn_code=${res}
-          "-Dbin_dir=${RunCMake_TEST_BINARY_DIR}"
-          -P "${RunCMake_SOURCE_DIR}/PreTestError.cmake"
-        )
-      return()
+      run_cmake_command(${TEST_NAME}-Build "${CMAKE_COMMAND}" --build "${RunCMake_TEST_BINARY_DIR}")
     endif()
 
     # execute cpack
diff --git a/Tests/RunCMake/CPack/PreTestError.cmake b/Tests/RunCMake/CPack/PreTestError.cmake
deleted file mode 100644
index f88f2e8..0000000
--- a/Tests/RunCMake/CPack/PreTestError.cmake
+++ /dev/null
@@ -1,7 +0,0 @@
-file(READ "${bin_dir}/test_output.txt" output)
-file(READ "${bin_dir}/test_error.txt" error)
-
-message(FATAL_ERROR "Error in pre-test phase '${RunCMake_TEST_STEP}'!\n"
-        "Return code: '${return_code}'\n"
-        "Info output: '${output}'\n"
-        "Error output: '${error}'")

https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=981068b79f3703a095f5aa254da6c7d3f997a265
commit 981068b79f3703a095f5aa254da6c7d3f997a265
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Thu Oct 1 09:38:19 2015 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Thu Oct 1 09:47:41 2015 -0400

    Tests: Avoid OS X 10.5 limitation warning in RunCMake.CPack* tests
    
    The DEPENDENCIES test case uses install(TARGETS) and so generates a warning:
    
     CMake Warning in CMakeLists.txt:
       WARNING: Target "test_prog" has runtime paths which cannot be changed
       during install.  To change runtime paths, OS X version 10.6 or newer is
       required.  Therefore, runtime paths will not be changed when installing.
       CMAKE_BUILD_WITH_INSTALL_RPATH may be used to work around this limitation.
    
    Set CMAKE_BUILD_WITH_INSTALL_RPATH to avoid the warning since we do not
    need to run the binaries from the build tree anyway.

diff --git a/Tests/RunCMake/CPack/DEPENDENCIES.cmake b/Tests/RunCMake/CPack/DEPENDENCIES.cmake
index 8a0ba35..4f6d65f 100644
--- a/Tests/RunCMake/CPack/DEPENDENCIES.cmake
+++ b/Tests/RunCMake/CPack/DEPENDENCIES.cmake
@@ -1,3 +1,5 @@
+set(CMAKE_BUILD_WITH_INSTALL_RPATH 1)
+
 file(WRITE "${CMAKE_CURRENT_BINARY_DIR}/test_lib.hpp"
     "int test_lib();\n")
 file(WRITE "${CMAKE_CURRENT_BINARY_DIR}/test_lib.cpp"

https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=e8954d0b3b6f7b29a00ec0238fc91d84021f0705
commit e8954d0b3b6f7b29a00ec0238fc91d84021f0705
Author:     Brad King <brad.king at kitware.com>
AuthorDate: Thu Oct 1 09:27:02 2015 -0400
Commit:     Brad King <brad.king at kitware.com>
CommitDate: Thu Oct 1 09:47:41 2015 -0400

    Tests: Add trailing newlines to generated sources in RunCMake.CPack_* tests
    
    This avoids compiler warnings on stderr while building them.

diff --git a/Tests/RunCMake/CPack/DEPENDENCIES.cmake b/Tests/RunCMake/CPack/DEPENDENCIES.cmake
index 0aef925..8a0ba35 100644
--- a/Tests/RunCMake/CPack/DEPENDENCIES.cmake
+++ b/Tests/RunCMake/CPack/DEPENDENCIES.cmake
@@ -1,11 +1,11 @@
 file(WRITE "${CMAKE_CURRENT_BINARY_DIR}/test_lib.hpp"
-    "int test_lib();")
+    "int test_lib();\n")
 file(WRITE "${CMAKE_CURRENT_BINARY_DIR}/test_lib.cpp"
-    "#include \"test_lib.hpp\"\nint test_lib() {return 0;}")
+    "#include \"test_lib.hpp\"\nint test_lib() {return 0;}\n")
 add_library(test_lib SHARED "${CMAKE_CURRENT_BINARY_DIR}/test_lib.cpp")
 
 file(WRITE "${CMAKE_CURRENT_BINARY_DIR}/main.cpp"
-    "#include \"test_lib.hpp\"\nint main() {return test_lib();}")
+    "#include \"test_lib.hpp\"\nint main() {return test_lib();}\n")
 add_executable(test_prog "${CMAKE_CURRENT_BINARY_DIR}/main.cpp")
 target_link_libraries(test_prog test_lib)
 

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

Summary of changes:


hooks/post-receive
-- 
CMake


More information about the Cmake-commits mailing list