Tests: Fix CMP0054 warnings

Hack the CMP0054 warning locally to be an error and run the test suite.
Resolve CMP0054 in Tests/* code as appropriate for each case.
This commit is contained in:
Brad King 2014-09-16 09:05:22 -04:00
parent 3b5ea54a27
commit fa7d47ac5c
8 changed files with 11 additions and 11 deletions

View File

@ -20,13 +20,13 @@ endif ()
string(TOUPPER "${MAJOR_TEST_MODULE}" MODULE_UPPER) string(TOUPPER "${MAJOR_TEST_MODULE}" MODULE_UPPER)
if ( ( ${MAJOR_TEST_MODULE}_FOUND OR ${MODULE_UPPER}_FOUND ) AND "${VERSION_VAR}") if ( ( ${MAJOR_TEST_MODULE}_FOUND OR ${MODULE_UPPER}_FOUND ) AND ${VERSION_VAR})
message(STATUS "${VERSION_VAR} is '${${VERSION_VAR}}'") message(STATUS "${VERSION_VAR} is '${${VERSION_VAR}}'")
if ("${VERSION_VAR}" VERSION_LESS MAJOR_TEST_VERSION) if (${VERSION_VAR} VERSION_LESS MAJOR_TEST_VERSION)
message(SEND_ERROR "Found version ${${VERSION_VAR}} is less than requested major version ${MAJOR_TEST_VERSION}") message(SEND_ERROR "Found version ${${VERSION_VAR}} is less than requested major version ${MAJOR_TEST_VERSION}")
endif () endif ()
math(EXPR V_PLUS_ONE "${MAJOR_TEST_VERSION} + 1") math(EXPR V_PLUS_ONE "${MAJOR_TEST_VERSION} + 1")
if ("${VERSION_VAR}" VERSION_GREATER V_PLUS_ONE) if (${VERSION_VAR} VERSION_GREATER V_PLUS_ONE)
message(SEND_ERROR "Found version ${${VERSION_VAR}} is greater than requested major version ${MAJOR_TEST_VERSION}") message(SEND_ERROR "Found version ${${VERSION_VAR}} is greater than requested major version ${MAJOR_TEST_VERSION}")
endif () endif ()
endif () endif ()

View File

@ -92,7 +92,7 @@ endfunction()
STRANGE_FUNCTION(var) STRANGE_FUNCTION(var)
set(second_var "second_var") set(second_var "second_var")
if("${var}" STREQUAL "strange_function" AND "${second_var}" STREQUAL "second_var") if("x${var}" STREQUAL "xstrange_function" AND "x${second_var}" STREQUAL "xsecond_var")
PASS("Case Test" "(${var} ${second_var})") PASS("Case Test" "(${var} ${second_var})")
else() else()
FAILED("Case test" "(${var} ${second_var})") FAILED("Case test" "(${var} ${second_var})")

View File

@ -36,7 +36,7 @@ macro(strange_macro m)
endmacro() endmacro()
STRANGE_MACRO(var) STRANGE_MACRO(var)
set(second_var "second_var") set(second_var "second_var")
if("${var}" STREQUAL "strange_macro" AND "${second_var}" STREQUAL "second_var") if("x${var}" STREQUAL "xstrange_macro" AND "x${second_var}" STREQUAL "xsecond_var")
PASS("Case Test" "(${var} ${second_var})") PASS("Case Test" "(${var} ${second_var})")
else() else()
FAILED("Case test" "(${var} ${second_var})") FAILED("Case test" "(${var} ${second_var})")

View File

@ -26,7 +26,7 @@ endif()
pkg_check_modules(FOO "${expected_path}") pkg_check_modules(FOO "${expected_path}")
if(NOT "FOO_FOUND") if(NOT FOO_FOUND)
message(FATAL_ERROR "Expected PKG_CONFIG_PATH: \"${expected_path}\".") message(FATAL_ERROR "Expected PKG_CONFIG_PATH: \"${expected_path}\".")
endif() endif()
@ -36,6 +36,6 @@ set(PKG_CONFIG_USE_CMAKE_PREFIX_PATH TRUE)
pkg_check_modules(BAR "${expected_path}" NO_CMAKE_PATH NO_CMAKE_ENVIRONMENT_PATH) pkg_check_modules(BAR "${expected_path}" NO_CMAKE_PATH NO_CMAKE_ENVIRONMENT_PATH)
if(NOT "BAR_FOUND") if(NOT BAR_FOUND)
message(FATAL_ERROR "Expected PKG_CONFIG_PATH: \"${expected_path}\".") message(FATAL_ERROR "Expected PKG_CONFIG_PATH: \"${expected_path}\".")
endif() endif()

View File

@ -46,6 +46,6 @@ endif()
pkg_check_modules(FOO "${expected_path}") pkg_check_modules(FOO "${expected_path}")
if(NOT "FOO_FOUND") if(NOT FOO_FOUND)
message(FATAL_ERROR "Expected PKG_CONFIG_PATH: \"${expected_path}\".") message(FATAL_ERROR "Expected PKG_CONFIG_PATH: \"${expected_path}\".")
endif() endif()

View File

@ -46,6 +46,6 @@ endif()
pkg_check_modules(FOO "${expected_path}" NO_CMAKE_ENVIRONMENT_PATH) pkg_check_modules(FOO "${expected_path}" NO_CMAKE_ENVIRONMENT_PATH)
if(NOT "FOO_FOUND") if(NOT FOO_FOUND)
message(FATAL_ERROR "Expected PKG_CONFIG_PATH: \"${expected_path}\".") message(FATAL_ERROR "Expected PKG_CONFIG_PATH: \"${expected_path}\".")
endif() endif()

View File

@ -46,6 +46,6 @@ endif()
pkg_check_modules(FOO "${expected_path}" NO_CMAKE_PATH) pkg_check_modules(FOO "${expected_path}" NO_CMAKE_PATH)
if(NOT "FOO_FOUND") if(NOT FOO_FOUND)
message(FATAL_ERROR "Expected PKG_CONFIG_PATH: \"${expected_path}\".") message(FATAL_ERROR "Expected PKG_CONFIG_PATH: \"${expected_path}\".")
endif() endif()

View File

@ -123,7 +123,7 @@ string(STRIP "ST2 " ST2)
string(STRIP " ST3" ST3) string(STRIP " ST3" ST3)
foreach(var ST1 ST2 ST3) foreach(var ST1 ST2 ST3)
if("${var}" STREQUAL "${${var}}") if("x${var}" STREQUAL "x${${var}}")
message("[${var}] == [${${var}}]") message("[${var}] == [${${var}}]")
else() else()
message(SEND_ERROR "Problem with the STRIP command for ${var}: [${${var}}]") message(SEND_ERROR "Problem with the STRIP command for ${var}: [${${var}}]")