Merge pull request #1 from cyanidle/main

Disable sanitizers by default
NEOLANT-Service · Nov 13, 2024 · a76c342 · a76c342
2 parents 267af81 + 88110a0
commit a76c342
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -32,7 +32,7 @@ set(CMAKE_CXX_EXTENSIONS OFF)
 
 option(BUILD_SHARED_LIBS         "Build libs as shared"         OFF)
 option(RPCXX_WITH_CODEGEN        "Compile Code Generator"       ON)
-option(RPCXX_TEST_SANITIZERS     "Enable sanitizers"            ON)
+option(RPCXX_TEST_SANITIZERS     "Enable sanitizers"            OFF)
 option(RPCXX_TEST_RPS            "Build QT5 based RPS test"     OFF)
 option(RPCXX_WITH_TESTS          "Build tests"                  OFF)
 

diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
@@ -22,6 +22,7 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
 ]]
 
+enable_testing()
 
 add_library(rpcxx-test-deps INTERFACE)
 if(RPCXX_TEST_SANITIZERS AND NOT MSVC)
@@ -39,7 +40,7 @@ endif()
 function(do_register target source)
     add_executable(${target} ${source})
     target_link_libraries(${target} PUBLIC rpcxx-test-deps)
-    add_test(NAME ${target} COMMAND ${target})
+    add_test(NAME ${target} COMMAND $<TARGET_FILE:${target}>)
 endfunction()
 
 do_register(rpcxx-rpc-test rpcxx/rpc_test.cpp)