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  1fe432a4a94ef20f8fad1b516a04541ba00e2610 (commit)
       via  72c29b28286a37cde40e1b231d915f2b1fd249f8 (commit)
       via  28fe71051162e95fe5eb116c11cf36eacc3b1d7d (commit)
      from  b278af8e08066537fa4eab38b3001e1c0df9f8d8 (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=1fe432a4a94ef20f8fad1b516a04541ba00e2610
commit 1fe432a4a94ef20f8fad1b516a04541ba00e2610
Merge: b278af8 72c29b2
Author:     Brad King <brad.k...@kitware.com>
AuthorDate: Fri Jan 20 15:50:10 2017 -0500
Commit:     CMake Topic Stage <kwro...@kitware.com>
CommitDate: Fri Jan 20 15:50:10 2017 -0500

    Merge topic 'sanitizer-options' into next
    
    72c29b28 ctest_memcheck: Fix sanitizers when MemoryCheckSanitizerOptions is 
empty
    28fe7105 Tests: Use CTEST_MEMORYCHECK_SANITIZER_OPTIONS where appropriate


https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=72c29b28286a37cde40e1b231d915f2b1fd249f8
commit 72c29b28286a37cde40e1b231d915f2b1fd249f8
Author:     Jamie Snape <jamie.sn...@kitware.com>
AuthorDate: Fri Jan 20 15:21:40 2017 -0500
Commit:     Brad King <brad.k...@kitware.com>
CommitDate: Fri Jan 20 15:49:15 2017 -0500

    ctest_memcheck: Fix sanitizers when MemoryCheckSanitizerOptions is empty

diff --git a/Source/CTest/cmCTestMemCheckHandler.cxx 
b/Source/CTest/cmCTestMemCheckHandler.cxx
index a062e64..beaef81 100644
--- a/Source/CTest/cmCTestMemCheckHandler.cxx
+++ b/Source/CTest/cmCTestMemCheckHandler.cxx
@@ -607,9 +607,13 @@ bool cmCTestMemCheckHandler::InitializeMemoryChecking()
       this->MemoryTesterDynamicOptions.push_back("-E");
       this->MemoryTesterDynamicOptions.push_back("env");
       std::string envVar;
-      std::string extraOptions = ":" +
-        this->CTest->GetCTestConfiguration("MemoryCheckSanitizerOptions");
+      std::string extraOptions;
       std::string suppressionsOption;
+      if (!this->CTest->GetCTestConfiguration("MemoryCheckSanitizerOptions")
+             .empty()) {
+        extraOptions = ":" +
+          this->CTest->GetCTestConfiguration("MemoryCheckSanitizerOptions");
+      }
       if (!this->CTest->GetCTestConfiguration("MemoryCheckSuppressionFile")
              .empty()) {
         suppressionsOption = ":suppressions=" +
@@ -632,7 +636,7 @@ bool cmCTestMemCheckHandler::InitializeMemoryChecking()
         envVar = "UBSAN_OPTIONS";
       }
       std::string outputFile =
-        envVar + "=log_path=\"" + this->MemoryTesterOutputFile + "\"";
+        envVar + "=log_path=" + this->MemoryTesterOutputFile;
       this->MemoryTesterEnvironmentVariable =
         outputFile + suppressionsOption + extraOptions;
       break;
diff --git a/Tests/RunCMake/ctest_memcheck/testAddressLeakSanitizer.cmake 
b/Tests/RunCMake/ctest_memcheck/testAddressLeakSanitizer.cmake
index 2a57b11..460d7cb 100644
--- a/Tests/RunCMake/ctest_memcheck/testAddressLeakSanitizer.cmake
+++ b/Tests/RunCMake/ctest_memcheck/testAddressLeakSanitizer.cmake
@@ -2,7 +2,7 @@
 # options
 
 message("ASAN_OPTIONS = [$ENV{ASAN_OPTIONS}]")
-string(REGEX REPLACE ".*log_path=\"([^\"]*)\".*" "\\1" LOG_FILE 
"$ENV{ASAN_OPTIONS}")
+string(REGEX REPLACE ".*log_path=([^:]*).*" "\\1" LOG_FILE 
"$ENV{ASAN_OPTIONS}")
 message("LOG_FILE=[${LOG_FILE}]")
 
 # if we are not asked to simulate AddressSanitizer don't do it
diff --git a/Tests/RunCMake/ctest_memcheck/testAddressSanitizer.cmake 
b/Tests/RunCMake/ctest_memcheck/testAddressSanitizer.cmake
index 3082e4b..f14a773 100644
--- a/Tests/RunCMake/ctest_memcheck/testAddressSanitizer.cmake
+++ b/Tests/RunCMake/ctest_memcheck/testAddressSanitizer.cmake
@@ -2,7 +2,7 @@
 # options
 
 message("ASAN_OPTIONS = [$ENV{ASAN_OPTIONS}]")
-string(REGEX REPLACE ".*log_path=\"([^\"]*)\".*" "\\1" LOG_FILE 
"$ENV{ASAN_OPTIONS}")
+string(REGEX REPLACE ".*log_path=([^:]*).*" "\\1" LOG_FILE 
"$ENV{ASAN_OPTIONS}")
 message("LOG_FILE=[${LOG_FILE}]")
 
 # if we are not asked to simulate address sanitizer don't do it
diff --git a/Tests/RunCMake/ctest_memcheck/testLeakSanitizer.cmake 
b/Tests/RunCMake/ctest_memcheck/testLeakSanitizer.cmake
index af214c8..272130e 100644
--- a/Tests/RunCMake/ctest_memcheck/testLeakSanitizer.cmake
+++ b/Tests/RunCMake/ctest_memcheck/testLeakSanitizer.cmake
@@ -2,7 +2,7 @@
 # options
 
 message("LSAN_OPTIONS = [$ENV{LSAN_OPTIONS}]")
-string(REGEX REPLACE ".*log_path=\"([^\"]*)\".*" "\\1" LOG_FILE 
"$ENV{LSAN_OPTIONS}")
+string(REGEX REPLACE ".*log_path=([^:]*).*" "\\1" LOG_FILE 
"$ENV{LSAN_OPTIONS}")
 message("LOG_FILE=[${LOG_FILE}]")
 
 # if we are not asked to simulate LeakSanitizer don't do it
diff --git a/Tests/RunCMake/ctest_memcheck/testMemorySanitizer.cmake 
b/Tests/RunCMake/ctest_memcheck/testMemorySanitizer.cmake
index c87af9a..3fc9335 100644
--- a/Tests/RunCMake/ctest_memcheck/testMemorySanitizer.cmake
+++ b/Tests/RunCMake/ctest_memcheck/testMemorySanitizer.cmake
@@ -1,8 +1,8 @@
-# this file simulates a program that has been built with thread sanitizer
+# this file simulates a program that has been built with MemorySanitizer
 # options
 
 message("MSAN_OPTIONS = [$ENV{MSAN_OPTIONS}]")
-string(REGEX REPLACE ".*log_path=\"([^\"]*)\".*" "\\1" LOG_FILE 
"$ENV{MSAN_OPTIONS}")
+string(REGEX REPLACE ".*log_path=([^:]*).*" "\\1" LOG_FILE 
"$ENV{MSAN_OPTIONS}")
 message("LOG_FILE=[${LOG_FILE}]")
 
 # if we are not asked to simulate address sanitizer don't do it
diff --git a/Tests/RunCMake/ctest_memcheck/testThreadSanitizer.cmake 
b/Tests/RunCMake/ctest_memcheck/testThreadSanitizer.cmake
index d591931..18208e2 100644
--- a/Tests/RunCMake/ctest_memcheck/testThreadSanitizer.cmake
+++ b/Tests/RunCMake/ctest_memcheck/testThreadSanitizer.cmake
@@ -1,8 +1,8 @@
-# this file simulates a program that has been built with thread sanitizer
+# this file simulates a program that has been built with ThreadSanitizer
 # options
 
 message("TSAN_OPTIONS = [$ENV{TSAN_OPTIONS}]")
-string(REGEX REPLACE ".*log_path=\"([^\"]*)\".*" "\\1" LOG_FILE 
"$ENV{TSAN_OPTIONS}")
+string(REGEX REPLACE ".*log_path=([^:]*).*" "\\1" LOG_FILE 
"$ENV{TSAN_OPTIONS}")
 message("LOG_FILE=[${LOG_FILE}]")
 
 set(error_types
diff --git a/Tests/RunCMake/ctest_memcheck/testUndefinedBehaviorSanitizer.cmake 
b/Tests/RunCMake/ctest_memcheck/testUndefinedBehaviorSanitizer.cmake
index 8ef3c0a..3c341a4 100644
--- a/Tests/RunCMake/ctest_memcheck/testUndefinedBehaviorSanitizer.cmake
+++ b/Tests/RunCMake/ctest_memcheck/testUndefinedBehaviorSanitizer.cmake
@@ -1,8 +1,8 @@
-# this file simulates a program that has been built with undefined behavior
-# sanitizer options
+# this file simulates a program that has been built with
+# UndefinedBehaviorSanitizer options
 
 message("UBSAN_OPTIONS = [$ENV{UBSAN_OPTIONS}]")
-string(REGEX REPLACE ".*log_path=\"([^\"]*)\".*" "\\1" LOG_FILE 
"$ENV{UBSAN_OPTIONS}")
+string(REGEX REPLACE ".*log_path=([^:]*).*" "\\1" LOG_FILE 
"$ENV{UBSAN_OPTIONS}")
 message("LOG_FILE=[${LOG_FILE}]")
 
 # if we are not asked to simulate address sanitizer don't do it

https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=28fe71051162e95fe5eb116c11cf36eacc3b1d7d
commit 28fe71051162e95fe5eb116c11cf36eacc3b1d7d
Author:     Jamie Snape <jamie.sn...@kitware.com>
AuthorDate: Fri Jan 20 15:21:16 2017 -0500
Commit:     Brad King <brad.k...@kitware.com>
CommitDate: Fri Jan 20 15:48:27 2017 -0500

    Tests: Use CTEST_MEMORYCHECK_SANITIZER_OPTIONS where appropriate
    
    Previously the ThreadSanitizer and MemorySanitizer tests erroneously
    used `CTEST_MEMORYCHECK_COMMAND_OPTIONS`.

diff --git a/Tests/RunCMake/ctest_memcheck/RunCMakeTest.cmake 
b/Tests/RunCMake/ctest_memcheck/RunCMakeTest.cmake
index 352a381..ab4c5ab 100644
--- a/Tests/RunCMake/ctest_memcheck/RunCMakeTest.cmake
+++ b/Tests/RunCMake/ctest_memcheck/RunCMakeTest.cmake
@@ -19,7 +19,7 @@ unset(CMAKELISTS_EXTRA_CODE)
 #-----------------------------------------------------------------------------
 # add ThreadSanitizer test
 set(CTEST_EXTRA_CODE
-"set(CTEST_MEMORYCHECK_COMMAND_OPTIONS \"report_bugs=1 history_size=5 
exitcode=55\")
+"set(CTEST_MEMORYCHECK_SANITIZER_OPTIONS 
\"report_bugs=1:history_size=5:exitcode=55\")
 ")
 set(CMAKELISTS_EXTRA_CODE
 "add_test(NAME TestSan COMMAND \"\${CMAKE_COMMAND}\"
@@ -71,7 +71,7 @@ unset(CTEST_EXTRA_CODE)
 #-----------------------------------------------------------------------------
 # add MemorySanitizer test
 set(CTEST_EXTRA_CODE
-"set(CTEST_MEMORYCHECK_COMMAND_OPTIONS 
\"simulate_sanitizer=1:report_bugs=1:history_size=5:exitcode=55\")
+"set(CTEST_MEMORYCHECK_SANITIZER_OPTIONS 
\"simulate_sanitizer=1:report_bugs=1:history_size=5:exitcode=55\")
 ")
 set(CMAKELISTS_EXTRA_CODE
 "add_test(NAME TestSan COMMAND \"\${CMAKE_COMMAND}\"

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

Summary of changes:
 Source/CTest/cmCTestMemCheckHandler.cxx                      |   10 +++++++---
 Tests/RunCMake/ctest_memcheck/RunCMakeTest.cmake             |    4 ++--
 Tests/RunCMake/ctest_memcheck/testAddressLeakSanitizer.cmake |    2 +-
 Tests/RunCMake/ctest_memcheck/testAddressSanitizer.cmake     |    2 +-
 Tests/RunCMake/ctest_memcheck/testLeakSanitizer.cmake        |    2 +-
 Tests/RunCMake/ctest_memcheck/testMemorySanitizer.cmake      |    4 ++--
 Tests/RunCMake/ctest_memcheck/testThreadSanitizer.cmake      |    4 ++--
 .../ctest_memcheck/testUndefinedBehaviorSanitizer.cmake      |    6 +++---
 8 files changed, 19 insertions(+), 15 deletions(-)


hooks/post-receive
-- 
CMake
_______________________________________________
Cmake-commits mailing list
Cmake-commits@cmake.org
http://public.kitware.com/mailman/listinfo/cmake-commits

Reply via email to