Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
cmake: stop patching tesh in FindValgrind, that's too ugly
authorMartin Quinson <martin.quinson@loria.fr>
Tue, 17 Jul 2018 21:18:13 +0000 (23:18 +0200)
committerMartin Quinson <martin.quinson@loria.fr>
Tue, 17 Jul 2018 21:18:13 +0000 (23:18 +0200)
tools/cmake/Modules/FindValgrind.cmake
tools/cmake/Tests.cmake

index 4afd05b..e2e0ba6 100644 (file)
@@ -9,23 +9,3 @@ find_program(VALGRIND_EXE
   /usr
   )
 mark_as_advanced(VALGRIND_EXE)
-
-if(enable_memcheck)
-  if (NOT VALGRIND_EXE MATCHES "NOTFOUND")
-    execute_process(COMMAND ${VALGRIND_EXE} --version  OUTPUT_VARIABLE "VALGRIND_VERSION")
-    message(STATUS "Valgrind version: ${VALGRIND_VERSION}")
-
-    set(TESH_WRAPPER ${CMAKE_HOME_DIRECTORY}/tools/cmake/scripts/my_valgrind.pl)
-    set(TESH_WRAPPER ${TESH_WRAPPER}\ --trace-children=yes\ --trace-children-skip=/usr/bin/*,/bin/*\ --leak-check=full\ --show-reachable=yes\ --track-origins=no\ --read-var-info=no\ --num-callers=20\ --suppressions=${CMAKE_HOME_DIRECTORY}/tools/simgrid.supp\ )
-    if(enable_memcheck_xml)
-      SET(TESH_WRAPPER ${TESH_WRAPPER}\ --xml=yes\ --xml-file=memcheck_test_%p.memcheck\ --child-silent-after-fork=yes\ )
-    endif()
-
-#    message(STATUS "tesh wrapper: ${TESH_WRAPPER}")
-
-    mark_as_advanced(TESH_WRAPPER)
-  else()
-    set(enable_memcheck false)
-    message(STATUS "Error: Command valgrind not found --> enable_memcheck autoset to false.")
-  endif()
-endif()
index d0968ff..159fc16 100644 (file)
@@ -11,6 +11,24 @@ SET(TESH_COMMAND "${PYTHON_EXECUTABLE}" ${CMAKE_BINARY_DIR}/bin/tesh)
 
 IF(enable_memcheck)
   INCLUDE(FindValgrind)
+
+  if (NOT VALGRIND_EXE MATCHES "NOTFOUND")
+    execute_process(COMMAND ${VALGRIND_EXE} --version  OUTPUT_VARIABLE "VALGRIND_VERSION")
+    message(STATUS "Valgrind version: ${VALGRIND_VERSION}")
+
+    set(TESH_WRAPPER ${CMAKE_HOME_DIRECTORY}/tools/cmake/scripts/my_valgrind.pl)
+    set(TESH_WRAPPER ${TESH_WRAPPER}\ --trace-children=yes\ --trace-children-skip=/usr/bin/*,/bin/*\ --leak-check=full\ --show-reachable=yes\ --track-origins=no\ --read-var-info=no\ --num-callers=20\ --suppressions=${CMAKE_HOME_DIRECTORY}/tools/simgrid.supp\ )
+    if(enable_memcheck_xml)
+      SET(TESH_WRAPPER ${TESH_WRAPPER}\ --xml=yes\ --xml-file=memcheck_test_%p.memcheck\ --child-silent-after-fork=yes\ )
+    endif()
+
+#    message(STATUS "tesh wrapper: ${TESH_WRAPPER}")
+
+    mark_as_advanced(TESH_WRAPPER)
+  else()
+    set(enable_memcheck false)
+    message(STATUS "Error: Command valgrind not found --> enable_memcheck autoset to false.")
+  endif()
 ENDIF()
 
 #some tests may take forever on non futexes systems, using busy_wait with n cores < n workers