Skip to content

Commit 89d41b6

Browse files
committed
don't override hdf5_external cache var
1 parent a7c9b55 commit 89d41b6

File tree

4 files changed

+27
-22
lines changed

4 files changed

+27
-22
lines changed

VERSION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
4.2.9
1+
4.2.10

cmake/install.cmake

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -3,21 +3,25 @@
33
include(CMakePackageConfigHelpers)
44

55
configure_package_config_file(${CMAKE_CURRENT_LIST_DIR}/config.cmake.in
6-
${CMAKE_CURRENT_BINARY_DIR}/CMakeFiles/${PROJECT_NAME}-config.cmake
7-
INSTALL_DESTINATION cmake)
6+
${CMAKE_CURRENT_BINARY_DIR}/CMakeFiles/${PROJECT_NAME}-config.cmake
7+
INSTALL_DESTINATION cmake
8+
)
89

910
write_basic_package_version_file(
10-
${CMAKE_CURRENT_BINARY_DIR}/CMakeFiles/${PROJECT_NAME}-config-version.cmake
11-
COMPATIBILITY SameMinorVersion)
11+
${CMAKE_CURRENT_BINARY_DIR}/CMakeFiles/${PROJECT_NAME}-config-version.cmake
12+
COMPATIBILITY SameMinorVersion
13+
)
1214

1315
install(EXPORT ${PROJECT_NAME}-targets
14-
NAMESPACE ${PROJECT_NAME}::
15-
DESTINATION cmake)
16+
NAMESPACE ${PROJECT_NAME}::
17+
DESTINATION cmake
18+
)
1619

1720
install(FILES
18-
${CMAKE_CURRENT_BINARY_DIR}/CMakeFiles/${PROJECT_NAME}-config.cmake
19-
${CMAKE_CURRENT_BINARY_DIR}/CMakeFiles/${PROJECT_NAME}-config-version.cmake
20-
DESTINATION cmake)
21+
${CMAKE_CURRENT_BINARY_DIR}/CMakeFiles/${PROJECT_NAME}-config.cmake
22+
${CMAKE_CURRENT_BINARY_DIR}/CMakeFiles/${PROJECT_NAME}-config-version.cmake
23+
DESTINATION cmake
24+
)
2125

2226
# --- CPack
2327

@@ -37,6 +41,7 @@ string(REGEX REPLACE "\n" ";" _cpack_ignore ${_cpack_ignore})
3741
set(CPACK_SOURCE_IGNORE_FILES "${_cpack_ignore}")
3842

3943
install(FILES ${CPACK_RESOURCE_FILE_README} ${CPACK_RESOURCE_FILE_LICENSE}
40-
DESTINATION share/docs/${PROJECT_NAME})
44+
DESTINATION share/docs/${PROJECT_NAME}
45+
)
4146

4247
include(CPack)

cmake/options.cmake

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
1-
2-
option(hdf5_external "Build HDF5 library")
31
option(autobuild "auto-build HDF5 if missing/broken" on)
42
option(dev "developer mode")
53

6-
option(shaky "run shaky tests that may segfault since designed to fail" off)
74
option(matlab "check HDF5 file writes with Matlab" off)
85
option(concepts "conceptual testing, for devs only" off)
96

src/tests/CMakeLists.txt

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,9 @@ endif()
8080
find_package(Python3 COMPONENTS Interpreter)
8181
if(NOT DEFINED h5py_ok)
8282
execute_process(COMMAND ${Python3_EXECUTABLE} -c "import h5py"
83-
RESULT_VARIABLE h5py_code
84-
TIMEOUT 15)
83+
RESULT_VARIABLE h5py_code
84+
TIMEOUT 15
85+
)
8586
if(h5py_code EQUAL 0)
8687
set(h5py_ok true CACHE BOOL "h5py OK")
8788
else()
@@ -90,7 +91,8 @@ if(NOT DEFINED h5py_ok)
9091
endif()
9192

9293
add_test(NAME h5fortran:PythonShape
93-
COMMAND ${Python3_EXECUTABLE} ${CMAKE_CURRENT_SOURCE_DIR}/test_shape.py ${shape_file})
94+
COMMAND ${Python3_EXECUTABLE} ${CMAKE_CURRENT_SOURCE_DIR}/test_shape.py ${shape_file}
95+
)
9496

9597
set_tests_properties(h5fortran:PythonShape PROPERTIES
9698
REQUIRED_FILES ${shape_file}
@@ -109,11 +111,12 @@ if(Matlab_FOUND)
109111
add_test(NAME h5fortran:MatlabShape COMMAND ${Matlab_MAIN_PROGRAM} -batch ${matlab_cmd})
110112

111113
set_tests_properties(h5fortran:MatlabShape PROPERTIES
112-
TIMEOUT 90
113-
LABELS shaky
114-
DEPENDS h5fortran:shape
115-
REQUIRED_FILES ${shape_file}
116-
FIXTURES_REQUIRED h5lib)
114+
TIMEOUT 90
115+
LABELS shaky
116+
DEPENDS h5fortran:shape
117+
REQUIRED_FILES ${shape_file}
118+
FIXTURES_REQUIRED h5lib
119+
)
117120

118121
endif(Matlab_FOUND)
119122
endif(matlab)

0 commit comments

Comments
 (0)