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
precice: fix output and cmake #76937
Conversation
46b076e
to
b8d029d
Compare
this builds on master for me, what does this try to fix? all i can see is editing the cmake file |
b8d029d
to
082677f
Compare
It was building before but I messed up the correct installation paths for the cmake files, this is fixed now I also added a patch for a pr that i opened upstream(precice/precice#605 ) because other wise the cmake files include paths that are not avaliable in nixos and it will fail when including the lib |
082677f
to
bc9d1f7
Compare
@jtojnar all done |
still got build failures |
bc9d1f7
to
80079a1
Compare
fixed |
@GrahamcOfBorg build precice |
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.
tested with nix-review
FYI we merged precice/precice#577 and precice/precice#609. |
Motivation for this change
Things done
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)Notify maintainers
cc @