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 HDF5 and add option to compile Fortran2003 interface #31781
Conversation
pkgs/tools/misc/hdf5/default.nix
Outdated
@@ -14,17 +15,20 @@ | |||
# (--enable-unsupported could be used to force the build) | |||
assert !cpp || mpi == null; | |||
|
|||
# Need a Fortran compiler for Fortran2003 bindings | |||
assert fortran2003 -> gfortran == null; |
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.
Shouldn't this be != null
?
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.
Whoops! Yes, it should.
pkgs/tools/misc/hdf5/default.nix
Outdated
@@ -45,6 +49,7 @@ stdenv.mkDerivation rec { | |||
configureFlags = [] | |||
++ optional cpp "--enable-cxx" | |||
++ optional (gfortran != null) "--enable-fortran" | |||
++ optional fortran2003 "--enable-fortran2003" |
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.
Why not just throw --enable-fortran2003
in with --enable-fortran
?
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.
Users might want to use the Fortran90 bindings instead of the Fortran2003 ones.
f987702
to
e63de7c
Compare
The doc states that:
I am fine with updating to 1.10 branch as default, but I think it would be preferable to keep the 1.8 branch around since it is still widely used (and default in many distributions). |
@lsix that's a good point, but I am not sure how to accomplish it... |
@robertodr just copy the old version of the
in This is how I would do it. Users should then use the override mechanism to use the 1.8 branch on their systems. |
Thanks for the explanation @lsix! |
Last time I checked, this bump broke several packages. The following packages seem to build:
|
Can you squash commits per expression. |
3ce479e
to
eb43cc5
Compare
Motivation for this change
The HDF5 package is updated from 1.8.18 to 1.10.1, as recommended on the HDF5 group website.
In addition, the possibility to compile the Fortran2003 interface is exposed via the
fortran2003
option for the 1.8.19 version of the package.Things done
hdf5_1_8
, updated from 1.8.18 to 1.8.19, to still provide the older version of HDF5.fortran2003
option tohdf5_1_8
. This allows compiling the Fortran2003 bindings, by passing--enable-fortran2003
to the configure script. This option was removed from 1.10.1, for which--enable-fortran
suffices to get the newer bindings compiled.build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)