Skip to content

Commit 5eada76

Browse files
committed
lint
1 parent b9e117f commit 5eada76

File tree

7 files changed

+18
-13
lines changed

7 files changed

+18
-13
lines changed

CMakeLists.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,6 @@ endif()
3838
find_package(MPI COMPONENTS C Fortran REQUIRED)
3939

4040

41-
include(cmake/gcd.cmake) # math functions
42-
4341
# --- external libraries: HDF5
4442

4543
include(cmake/check_mpi.cmake)
File renamed without changes.

cmake/Modules/CodeCoverage.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ if(NOT (CMAKE_BUILD_TYPE STREQUAL "Debug" OR GENERATOR_IS_MULTI_CONFIG))
201201
endif() # NOT (CMAKE_BUILD_TYPE STREQUAL "Debug" OR GENERATOR_IS_MULTI_CONFIG)
202202

203203
if(CMAKE_C_COMPILER_ID STREQUAL "GNU" OR CMAKE_Fortran_COMPILER_ID STREQUAL "GNU")
204-
link_libraries(gcov)
204+
link_libraries(gcov)
205205
endif()
206206

207207
# Defines a target for running and collection code coverage information

cmake/abi_check/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
cmake_minimum_required(VERSION 3.19...3.23)
1+
cmake_minimum_required(VERSION 3.19)
22
project(abi_check LANGUAGES C Fortran)
33

44
enable_testing()

cmake/config.cmake.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ include(CMakeFindDependencyMacro)
44

55
list(APPEND CMAKE_MODULE_PATH ${CMAKE_CURRENT_LIST_DIR})
66

7-
find_dependency(HDF5 1.8.7 COMPONENTS Fortran parallel)
7+
find_dependency(HDF5 COMPONENTS Fortran parallel)
88

99
find_dependency(MPI COMPONENTS C Fortran)
1010

cmake/options.cmake

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@ option(ENABLE_COVERAGE "Code coverage tests")
44

55
option(ourFindMPI "Use our FindMPI")
66

7+
message(STATUS "${PROJECT_NAME} ${PROJECT_VERSION} CMake ${CMAKE_VERSION}")
8+
79
set(CMAKE_TLS_VERIFY true)
810

911

@@ -19,12 +21,6 @@ endif()
1921

2022
cmake_path(SET CMAKE_MODULE_PATH ${CMAKE_CURRENT_LIST_DIR}/Modules)
2123

22-
if(CMAKE_INSTALL_PREFIX_INITIALIZED_TO_DEFAULT)
23-
# will not take effect without FORCE
24-
# CMAKE_BINARY_DIR in case it's used from FetchContent
25-
set(CMAKE_INSTALL_PREFIX ${CMAKE_BINARY_DIR} CACHE PATH "Install top-level directory" FORCE)
26-
endif()
27-
2824
# Rpath options necessary for shared library install to work correctly in user projects
2925
set(CMAKE_INSTALL_NAME_DIR ${CMAKE_INSTALL_PREFIX}/lib)
3026
set(CMAKE_INSTALL_RPATH ${CMAKE_INSTALL_PREFIX}/lib)
@@ -33,6 +29,17 @@ set(CMAKE_INSTALL_RPATH_USE_LINK_PATH true)
3329
# Necessary for shared library with Visual Studio / Windows oneAPI
3430
set(CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS true)
3531

32+
if(CMAKE_INSTALL_PREFIX_INITIALIZED_TO_DEFAULT)
33+
# will not take effect without FORCE
34+
# CMAKE_BINARY_DIR for use from FetchContent
35+
set(CMAKE_INSTALL_PREFIX ${CMAKE_BINARY_DIR} CACHE PATH "Install top-level directory" FORCE)
36+
endif()
37+
38+
# allow CMAKE_PREFIX_PATH with ~ expand
39+
if(CMAKE_PREFIX_PATH)
40+
get_filename_component(CMAKE_PREFIX_PATH ${CMAKE_PREFIX_PATH} ABSOLUTE)
41+
endif()
42+
3643
# --- auto-ignore build directory
3744
if(NOT EXISTS ${PROJECT_BINARY_DIR}/.gitignore)
3845
file(WRITE ${PROJECT_BINARY_DIR}/.gitignore "*")

cmake/zlib.cmake

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ URL_HASH SHA256=${zlib_sha256}
4444
CMAKE_ARGS ${zlib_cmake_args}
4545
BUILD_BYPRODUCTS ${ZLIB_LIBRARIES}
4646
CONFIGURE_HANDLED_BY_BUILD ON
47-
INACTIVITY_TIMEOUT 15
47+
INACTIVITY_TIMEOUT 60
4848
)
4949
else()
5050
ExternalProject_Add(ZLIB
@@ -54,7 +54,7 @@ GIT_SHALLOW true
5454
CMAKE_ARGS ${zlib_cmake_args}
5555
BUILD_BYPRODUCTS ${ZLIB_LIBRARIES}
5656
CONFIGURE_HANDLED_BY_BUILD ON
57-
INACTIVITY_TIMEOUT 15
57+
INACTIVITY_TIMEOUT 60
5858
)
5959
endif()
6060

0 commit comments

Comments
 (0)