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
suitesparse: 5.7.2 -> 5.8.1 #98485
suitesparse: 5.7.2 -> 5.8.1 #98485
Conversation
pkgs/development/libraries/science/math/suitesparse/default.nix
Outdated
Show resolved
Hide resolved
44908a7
to
3695953
Compare
3695953
to
b6d4976
Compare
This diff looks good to me, but it would be nice if the commit could be split. I assume that the gmp/mpfr dependencies and gnum4 removal come with the update. But it would be nice to put the BLAS-related changes in a separate commit to make it easier to inspect the history in the future. Also, I am not very familiar with this derivation and it has a lot of reverse-dependencies, so it's probably better if @jtojnar or @ttuegel reviewed this PR. |
Update dependencies. Remove not needed `BLAS` and `LAPACK` makeFlags for non darwin systems.
b6d4976
to
6bbfd17
Compare
Done.
I tested octave to build and run with this update, along with other related PRs merged (#98499 #98500 & #98501). I didn't test suitesparse related functionality though. |
Thanks! 😃 |
Assert that compatible lapack and blas implementations are used. Update dependencies. Remove now not needed
BLAS
andLAPACK
makeFlags.Motivation for this change
Things done
Tested that Octave and Sundials build with this version, in a dirty branch, not with nixpkgs-review.
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)