Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes finding HDF5 header #2867

Merged
merged 1 commit into from
Mar 1, 2024
Merged

Conversation

Julius-Plehn
Copy link
Contributor

Without this PR, only the first file within the loop is checked for. Therefore, it fails if the first file is not found.

Without PR:

-- Checking for HDF5 config header
-- Looking for H5public.h
-- Looking for H5public.h - not found
CMake Error at cmake/check_hdf5.cmake:30 (message):
  Could not include any HDF5 config headers
Call Stack (most recent call first):

With PR:

-- Looking for H5public.h
-- Looking for H5public.h - not found
-- Looking for H5pubconf.h
-- Looking for H5pubconf.h - found
-- Using H5pubconf.h to check for feature macros

@CLAassistant
Copy link

CLAassistant commented Feb 22, 2024

CLA assistant check
All committers have signed the CLA.

@WardF WardF merged commit 6662212 into Unidata:main Mar 1, 2024
103 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants