Skip to content

Commit

Permalink
Fixes GH#1027 compilation error (#3654)
Browse files Browse the repository at this point in the history
  • Loading branch information
brtnfld authored Oct 10, 2023
1 parent e4cea35 commit 0ffde9d
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 1 deletion.
6 changes: 6 additions & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -450,6 +450,12 @@ include (${HDF_RESOURCES_DIR}/ConfigureChecks.cmake)

set (CMAKE_INCLUDE_CURRENT_DIR_IN_INTERFACE ON)

#-----------------------------------------------------------------------------
# Include directories in the source or build tree should come before other
# directories to prioritize headers in the sources over installed ones.
#-----------------------------------------------------------------------------
set(CMAKE_INCLUDE_DIRECTORIES_PROJECT_BEFORE ON)

#-----------------------------------------------------------------------------
# Mac OS X Options
#-----------------------------------------------------------------------------
Expand Down
9 changes: 8 additions & 1 deletion release_docs/RELEASE.txt
Original file line number Diff line number Diff line change
Expand Up @@ -682,7 +682,14 @@ Bug Fixes since HDF5-1.14.0 release
Checks were added to the CMake and Autotools code to verify that CLOCK_MONOTONIC_COARSE,
PTHREAD_MUTEX_ADAPTIVE_NP and pthread_condattr_setclock() are available before attempting
to use them in Subfiling VFD-related utility code. Without these checks, attempting
to build the Subfiling VFD on macOS would fail.
to build the Subfiling VFD on macOS would fail.

- Fixes the ordering of INCLUDES when building with CMake

Include directories in the source or build tree should come before other
directories to prioritize headers in the sources over installed ones.

Fixes GitHub #1027

- The accum test now passes on macOS 12+ (Monterey) w/ CMake

Expand Down

0 comments on commit 0ffde9d

Please sign in to comment.