Skip to content

Commit 49a0c5e

Browse files
committed
[CMake] Error if SOFIE test requirements not found and fail-on-missing
It's not good that we only emit a warning if SOFIE is not tested. Like this, we risk losing test coverage without us knowing.
1 parent 684aa04 commit 49a0c5e

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

cmake/modules/SearchInstalledSoftware.cmake

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1609,7 +1609,11 @@ if(tmva-sofie)
16091609
message(STATUS "Looking for BLAS as an optional testing dependency of TMVA-SOFIE")
16101610
find_package(BLAS)
16111611
if(NOT BLAS_FOUND)
1612-
message(WARNING "BLAS not found: TMVA-SOFIE will not be fully tested")
1612+
if(fail-on-missing)
1613+
message(FATAL_ERROR "BLAS not found, but it's required for TMVA-SOFIE testing")
1614+
else()
1615+
message(WARNING "BLAS not found: TMVA-SOFIE will not be fully tested")
1616+
endif()
16131617
endif()
16141618
endif()
16151619
message(STATUS "Looking for Protobuf")

0 commit comments

Comments
 (0)