-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
hdf5 : update to 1.12.0 #25569
hdf5 : update to 1.12.0 #25569
Conversation
420060a
to
7a9f977
Compare
Please check the output of See instructions in https://www.hdfgroup.org/downloads/hdf5/ for potential fixes. |
FWIW, the test suite seems to be doing fine. |
ok. These are the packages :
I will check everyone. |
I see you have a few PRs related to this hdf5 update. please have them all on this branch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the update file has to be updated to something like this:
site="https://support.hdfgroup.org/ftp/HDF5/releases/ReleaseFiles/"
pattern='<a href="hdf5-\K[\d.]+(?=-RELEASE.txt">)'
srcpkgs/hdf5/template
Outdated
homepage="https://hdfgroup.org" | ||
distfiles="https://www.hdfgroup.org/ftp/HDF5/current/src/hdf5-${version}.tar.gz" | ||
checksum=6d4ce8bf902a97b050f6f491f4268634e252a63dadd6656a1a9be5b7b7726fa8 | ||
distfiles="https://support.hdfgroup.org/ftp/HDF5/releases/${pkgname}-${_version}/${pkgname}-${version}/src/${pkgname}-${version}.tar.gz" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you use ${version%.*}
instead of _version?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this could be a good time to add a changelog:
changelog="https://support.hdfgroup.org/ftp/HDF5/releases/hdf5-${version%.*}/hdf5-${version}/src/hdf5-${version}-RELEASE.txt"
med 4.1 does not build with hdf5 1.12
|
ok, then you close this pr. |
I think this PR could wait for the next version of med or create a new package to hdf110 with the newest version from the 1.10 series and make med depend on it. |
No description provided.