LLNL / Silo

Mesh and Field I/O Library and Scientific Database
https://silo.llnl.gov
Other
30 stars 24 forks source link

All tests tail because of path mismatch #288

Open yurivict opened 2 years ago

yurivict commented 2 years ago

tests/CMake/SiloMakeCheckRunner.cmake attempts to run test executables just by their name, but the current directory isn't in the path.

This patch solves this problem:

--- tests/CMake/SiloMakeCheckRunner.cmake.orig  2022-12-03 06:53:38 UTC
+++ tests/CMake/SiloMakeCheckRunner.cmake
@@ -72,7 +72,7 @@ function(silo_add_make_check_runner)
         message(WARNING "silo_add_make_check_runner: NAME argument is required.")
         return()
     endif()
-    set(test_cmd ${samcr_NAME})
+    set(test_cmd "./${samcr_NAME}")
     if("ARGS" IN_LIST samcr_KEYWORDS_MISSING_VALUES)
         message(WARNING "silo_add_make_check_runner: ARGS argument provided without a value.")
         return()

Did you only test the testsuite on Windows?

FreeBSD 13.1