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

Update HDF from 1.12.1 to 1.12.3 (2nd attempt) #4716

Closed
wants to merge 3 commits into from
Closed

Conversation

seanm
Copy link
Contributor

@seanm seanm commented Jun 10, 2024

No description provided.

seanm and others added 3 commits June 6, 2024 20:02
Code extracted from:

    https://github.com/HDFGroup/hdf5.git

at commit 106fb56936d1d785ca96147601a3608e433ddc3b (hdf5-1_12_3).
# By HDF5 Maintainers
* upstream-HDF5:
  HDF5 2023-11-06 (106fb569)

# Conflicts:
#	Modules/ThirdParty/HDF5/src/itkhdf5/CMakeFilters.cmake
#	Modules/ThirdParty/HDF5/src/itkhdf5/CMakeLists.txt
#	Modules/ThirdParty/HDF5/src/itkhdf5/config/cmake/ConfigureChecks.cmake
#	Modules/ThirdParty/HDF5/src/itkhdf5/config/cmake/HDFCompilerFlags.cmake
#	Modules/ThirdParty/HDF5/src/itkhdf5/config/cmake/HDFMacros.cmake
#	Modules/ThirdParty/HDF5/src/itkhdf5/config/cmake_ext_mod/ConfigureChecks.cmake
#	Modules/ThirdParty/HDF5/src/itkhdf5/config/cmake_ext_mod/HDFUseCXX.cmake
#	Modules/ThirdParty/HDF5/src/itkhdf5/hl/CMakeLists.txt
#	Modules/ThirdParty/HDF5/src/itkhdf5/src/CMakeLists.txt
@thewtex
Copy link
Member

thewtex commented Jun 20, 2024

Note: once CI is green, the symbols will also need to be per itk_hdf5_mangle.h.in.

@seanm
Copy link
Contributor Author

seanm commented Jun 21, 2024

Note: once CI is green, the symbols will also need to be per itk_hdf5_mangle.h.in.

Hopefully next week I'll have time to try this again, on top of 5.4 branch.

@hjmjohnson
Copy link
Member

Superceeded by #4940

@hjmjohnson hjmjohnson closed this Nov 11, 2024
@hjmjohnson hjmjohnson mentioned this pull request Nov 11, 2024
7 tasks
@blowekamp
Copy link
Member

Superceeded by #4940

What in particular to you want to ensure is in this PR? That other PR, was not building correctly, and this PR makes an effort to remove changes from the HDF5 code base.

@hjmjohnson
Copy link
Member

I just wanted to ensure that this PR references the other one as context since they both seem to be addressing the same update desire.

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.

4 participants