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
Add sundials #36526
Add sundials #36526
Conversation
Please change the commit message to |
}; | ||
|
||
preConfigure = '' | ||
export cmakeFlags="-DCMAKE_INSTALL_PREFIX=$out -DEXAMPLES_INSTALL_PATH=$out/share/examples $cmakeFlags" |
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.
You can say cmakeFlags = ...
as a top-level attribute rather than setting it in preConfigure I believe.
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.
Yes, that should be possible. See here for an example.
The example also has
|
Hah, should've looked further. You could try this out. Maybe it works now. |
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.
Please rebase this PR to a current version of master
to resolve the merge conflicts.
It seems I am not a maintainer? |
description = "Suite of nonlinear differential/algebraic equation solvers"; | ||
homepage = https://computation.llnl.gov/casc/sundials/main.html; | ||
platforms = platforms.all; | ||
maintainers = [ idontgetoutmuch ]; |
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.
You need to add an appropriate attribute in maintainers/maintainer-list.nix
and refer to that in the maintainers
field.
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.
Please note that you must refer to maintainers.idontgetoutmuch
here.
homepage = https://computation.llnl.gov/casc/sundials/main.html; | ||
platforms = platforms.all; | ||
maintainers = [ idontgetoutmuch ]; | ||
licencse = { url = https://computation.llnl.gov/projects/sundials/license; }; |
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.
Typo here: licencse
-> license
. Also that looks like a three-clause BSD license and thus can be referenced by licenses.bsd3
.
@peti what else do I need to do? |
Motivation for this change
A good package for solving differential equations that ought to be in nix.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)