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.

Sign up for GitHub

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

Jump to bottom

Fix configure error for hdf5 mpi C bindings #752

Merged
merged 1 commit into from
May 13, 2024
Merged

Fix configure error for hdf5 mpi C bindings #752

merged 1 commit into from
May 13, 2024

Conversation

streeve
Copy link
Member

@streeve streeve commented May 9, 2024 •
edited
Loading 

CMake Error at /github/home/hdf5/cmake/hdf5-targets.cmake:68 (set_target_properties):
  The link interface of target "hdf5::hdf5-shared" contains:

    MPI::MPI_C

  but the target was not found.

@streeve streeve requested a review from junghans May 9, 2024 16:04
@streeve streeve self-assigned this May 9, 2024
@streeve streeve force-pushed the fixup_hdf5_mpi_c branch from 5661118 to 93cb8e2 Compare May 9, 2024 17:01
@dalg24
Copy link
Collaborator

dalg24 commented May 9, 2024 •
edited
Loading

edited sorry commented at the wrong place

@streeve streeve requested review from dalg24, abisner and lebuller May 13, 2024 13:13
@streeve
Copy link
Member Author

streeve commented May 13, 2024

Docker/jenkins failures unrelated

@streeve streeve merged commit 411e344 into ECP-copa:master May 13, 2024
32 of 34 checks passed
@streeve streeve deleted the fixup_hdf5_mpi_c branch May 13, 2024 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lebuller lebuller lebuller approved these changes

@junghans junghans Awaiting requested review from junghans

@dalg24 dalg24 Awaiting requested review from dalg24

@abisner abisner Awaiting requested review from abisner

Assignees

@streeve streeve

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@streeve @dalg24 @lebuller