Skip to content

Commit a24081c

Browse files
committed
FindHDF5: remove unneeded aux var
1 parent f18d503 commit a24081c

File tree

1 file changed

+7
-13
lines changed

1 file changed

+7
-13
lines changed

cmake/Modules/FindHDF5.cmake

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -162,16 +162,14 @@ if(NOT ZLIB_ROOT)
162162
endif()
163163

164164

165-
set(hdf5_prereqs true PARENT_SCOPE)
166-
167165
if(hdf5_have_zlib)
166+
168167
if(HDF5_FIND_REQUIRED)
169168
find_package(ZLIB REQUIRED)
170169
else()
171170
find_package(ZLIB)
172171
endif()
173172
if(NOT ZLIB_FOUND)
174-
set(hdf5_prereqs false PARENT_SCOPE)
175173
return()
176174
endif()
177175

@@ -194,7 +192,6 @@ if(hdf5_have_zlib)
194192
)
195193

196194
if(NOT SZIP_LIBRARY AND SZIP_INCLUDE_DIR)
197-
set(hdf5_prereqs false PARENT_SCOPE)
198195
return()
199196
endif()
200197

@@ -511,7 +508,6 @@ else()
511508
find_program(HDF5_Fortran_COMPILER_EXECUTABLE
512509
NAMES ${wrapper_names}
513510
NAMES_PER_DIR
514-
HINTS ${HOMEBREW_PREFIX} ENV HOMEBREW_PREFIX ${MACPORTS_PREFIX} ENV MACPORTS_PREFIX
515511
PATHS ${hdf5_binpref}
516512
PATH_SUFFIXES ${hdf5_binsuf}
517513
)
@@ -567,7 +563,6 @@ else()
567563
find_program(HDF5_CXX_COMPILER_EXECUTABLE
568564
NAMES ${wrapper_names}
569565
NAMES_PER_DIR
570-
HINTS ${HOMEBREW_PREFIX} ENV HOMEBREW_PREFIX ${MACPORTS_PREFIX} ENV MACPORTS_PREFIX
571566
PATHS ${hdf5_binpref}
572567
PATH_SUFFIXES ${hdf5_binsuf}
573568
)
@@ -617,7 +612,6 @@ else()
617612
find_program(HDF5_C_COMPILER_EXECUTABLE
618613
NAMES ${wrapper_names}
619614
NAMES_PER_DIR
620-
HINTS ${HOMEBREW_PREFIX} ENV HOMEBREW_PREFIX ${MACPORTS_PREFIX} ENV MACPORTS_PREFIX
621615
PATHS ${hdf5_binpref}
622616
PATH_SUFFIXES ${hdf5_binsuf}
623617
)
@@ -745,7 +739,7 @@ endfunction(check_fortran_links)
745739

746740
function(check_hdf5_link)
747741

748-
if(NOT (hdf5_prereqs AND HDF5_C_FOUND))
742+
if(NOT HDF5_C_FOUND)
749743
return()
750744
endif()
751745

@@ -832,7 +826,7 @@ elseif(CMAKE_SYSTEM_PROCESSOR MATCHES "(aarch64|arm64)")
832826
list(APPEND hdf5_isuf openmpi-aarch64 mpich-aarch64) # CentOS
833827
endif()
834828

835-
if(NOT HDF5_ROOT AND CMAKE_Fortran_COMPILER_ID STREQUAL GNU)
829+
if(NOT HDF5_ROOT AND CMAKE_Fortran_COMPILER_ID STREQUAL "GNU")
836830
# CentOS paths
837831
if(parallel IN_LIST HDF5_FIND_COMPONENTS)
838832
list(PREPEND hdf5_msuf gfortran/modules/openmpi gfortran/modules/mpich)
@@ -843,7 +837,7 @@ endif()
843837

844838
# --- binary prefix / suffix
845839
set(hdf5_binpref)
846-
if(CMAKE_SYSTEM_NAME STREQUAL Linux)
840+
if(CMAKE_SYSTEM_NAME STREQUAL "Linux")
847841
set(hdf5_binpref /usr/lib64)
848842
endif()
849843

@@ -866,11 +860,11 @@ if(HDF5_C_FOUND)
866860
detect_config()
867861
endif(HDF5_C_FOUND)
868862

869-
if(hdf5_prereqs AND HDF5_C_FOUND AND CXX IN_LIST HDF5_FIND_COMPONENTS)
863+
if(HDF5_C_FOUND AND CXX IN_LIST HDF5_FIND_COMPONENTS)
870864
find_hdf5_cxx()
871865
endif()
872866

873-
if(hdf5_prereqs AND HDF5_C_FOUND AND Fortran IN_LIST HDF5_FIND_COMPONENTS)
867+
if(HDF5_C_FOUND AND Fortran IN_LIST HDF5_FIND_COMPONENTS)
874868
find_hdf5_fortran()
875869
endif()
876870

@@ -887,7 +881,7 @@ list(REMOVE_ITEM CMAKE_IGNORE_PATH ${h5_ignore_path})
887881

888882
include(FindPackageHandleStandardArgs)
889883
find_package_handle_standard_args(HDF5
890-
REQUIRED_VARS HDF5_C_LIBRARIES HDF5_links hdf5_prereqs
884+
REQUIRED_VARS HDF5_C_LIBRARIES HDF5_links
891885
VERSION_VAR HDF5_VERSION
892886
HANDLE_COMPONENTS
893887
HANDLE_VERSION_RANGE

0 commit comments

Comments
 (0)