CMake >= 3.19 for robustness

HDF5 is complex enough without adding complexity from
ancient CMake need to test

Author: scivision

CMakeLists.txt

@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 3.14...3.28)
+cmake_minimum_required(VERSION 3.19...3.28)
 
 if(CMAKE_SOURCE_DIR STREQUAL CMAKE_BINARY_DIR)
   message(FATAL_ERROR "Please do out of source build like
@@ -12,7 +12,7 @@ endif()
 
 project(h5fortran
 LANGUAGES C Fortran
-VERSION 4.10.3
+VERSION 4.10.4
 )
 
 include(CTest)

cmake/CheckHDF5.cmake

@@ -1,8 +1,8 @@
 # HDF5 1.14.0 has a bug where basic types are not correct, which causes confusing runtime errors
 # check for those and other problems that may arise before build.
 
-include(CheckCSourceRuns)
-include(CheckFortranSourceRuns)
+include(CheckSourceRuns)
+
 
 function(check_hdf5_c)
 
@@ -36,7 +36,7 @@ return EXIT_SUCCESS;
 }
 ]=])
 
-check_c_source_runs("${_src}" hdf5_c_types)
+check_source_runs(C "${_src}" hdf5_c_types)
 
 if(NOT hdf5_c_types)
   message(FATAL_ERROR "HDF5 C types failed check")
@@ -82,7 +82,7 @@ end program
 ]=]
 )
 
-check_fortran_source_runs("${_src}" hdf5_fortran_types)
+check_source_runs(Fortran "${_src}" hdf5_fortran_types)
 
 if(NOT hdf5_fortran_types)
   message(FATAL_ERROR "HDF5 Fortran types failed check")

cmake/FindHDF5.cmake

@@ -60,8 +60,7 @@ Targets
 #]=======================================================================]
 
 include(CheckSymbolExists)
-include(CheckCSourceCompiles)
-include(CheckFortranSourceCompiles)
+include(CheckSourceCompiles)
 
 
 function(get_flags exec outvar)
@@ -696,7 +695,7 @@ else()
   ]=])
 endif(HDF5_parallel_FOUND)
 
-check_c_source_compiles("${src}" HDF5_C_links)
+check_source_compiles(C "${src}" HDF5_C_links)
 
 endfunction(check_c_links)
 
@@ -737,7 +736,7 @@ else()
   end program")
 endif()
 
-check_fortran_source_compiles(${src} HDF5_Fortran_links SRC_EXT f90)
+check_source_compiles(Fortran ${src} HDF5_Fortran_links)
 
 endfunction(check_fortran_links)
 

cmake/compilers.cmake

@@ -1,13 +1,13 @@
-include(CheckFortranSourceCompiles)
+include(CheckSourceCompiles)
 
 # --- not all platforms have ieee_arithmetic e.g. aarch64 GCC
-check_fortran_source_compiles("program a
+check_source_compiles(Fortran
+"program a
 use, intrinsic :: ieee_arithmetic, only : ieee_quiet_nan, ieee_value
 real :: NaN
 NaN = ieee_value(0., ieee_quiet_nan)
 end program"
 HAVE_IEEE_ARITH
-SRC_EXT f90
 )
 
 # --- C compile flags

options.cmake

@@ -10,9 +10,6 @@ option(concepts "conceptual testing, for devs only" off)
 
 option(CMAKE_TLS_VERIFY "Verify TLS certificates" on)
 
-include(GNUInstallDirs)
-
-
 if(BUILD_SHARED_LIBS AND MSVC)
   message(WARNING "Intel oneAPI has trouble with shared libs in general on Windows, try
     cmake -DBUILD_SHARED_LIBS=off")