Skip to content

Commit 2fd3989

Browse files
committed
ci: enable building with ASan on Windows for testing
With the merging of f5e500c, it's now possible to use AddressSanitizer on Windows with MSVC, so there's no reason not to use it when building the tests in CI.
1 parent f5e500c commit 2fd3989

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

.github/scripts/build-windows.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,13 +30,15 @@ else
3030
-G "Visual Studio 17 2022" \
3131
-DCMAKE_BUILD_TYPE="${BUILD_TYPE}" \
3232
-DENABLE_OPENMP="ON" \
33+
-DENABLE_SANITIZERS="ON" \
3334
-DCMAKE_GENERATOR_PLATFORM="Win32" \
3435
..
3536
fi
3637

3738
cmake --build . --config "${BUILD_TYPE}" -- -maxcpucount
3839

39-
bin/$BUILD_TYPE/solvespace-testsuite.exe
40+
# Run the tests in the proper environment (required for having the ASan libraries available)
41+
cmake --build . --config "${BUILD_TYPE}" -t test_solvespace -- -maxcpucount
4042

4143
if [ "$3" = "x64" ]; then
4244
if [ "$2" != "openmp" ]; then

0 commit comments

Comments
 (0)