Просмотр исходного кода

BUG: Should test HAVE_<VARIABLE> before repeating test, not just whether <VARIABLE> is set.

Brad King 23 лет назад
Родитель
Сommit
98d10a42d3
1 измененных файлов с 2 добавлено и 2 удалено
  1. 2 2
      Modules/CheckTypeSize.cmake

+ 2 - 2
Modules/CheckTypeSize.cmake

@@ -8,7 +8,7 @@
 
 MACRO(CHECK_TYPE_SIZE TYPE VARIABLE)
   SET(CMAKE_ALLOW_UNKNOWN_VARIABLE_READ_ACCESS 1)
-  IF("${VARIABLE}" MATCHES "^${VARIABLE}$")
+  IF("HAVE_${VARIABLE}" MATCHES "^HAVE_${VARIABLE}$")
     SET(MACRO_CHECK_TYPE_SIZE_FLAGS 
 	"-DCHECK_TYPE_SIZE_TYPE=\"${TYPE}\" ${CMAKE_REQUIRED_FLAGS}")
     IF(HAVE_SYS_TYPES_H)
@@ -37,6 +37,6 @@ MACRO(CHECK_TYPE_SIZE TYPE VARIABLE)
         "Determining size of ${TYPE} failed with the following output:\n${OUTPUT}\n"
         APPEND)
     ENDIF(HAVE_${VARIABLE})
-  ENDIF("${VARIABLE}" MATCHES "^${VARIABLE}$")
+  ENDIF("HAVE_${VARIABLE}" MATCHES "^HAVE_${VARIABLE}$")
   SET(CMAKE_ALLOW_UNKNOWN_VARIABLE_READ_ACCESS )
 ENDMACRO(CHECK_TYPE_SIZE)