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
givaro: Add darwin support #45001
givaro: Add darwin support #45001
Conversation
f27a310
to
d22d9aa
Compare
|
||
# needed for darwin | ||
doInstallCheck = true; | ||
installCheckTarget = "check"; |
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.
What do these do out of interest that makes it work?
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 tests are linked such that they look for the dylib file in the nix store, in the place where it will be installed. These make it so the tests get run after installPhase
, so the dylib is actually there, otherwise it segfaults because it can't find it. I'm not sure why this only happens on darwin.
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 add that explanation to the comment instead of "needed for darwin". Also explicitly set doCheck = false
to make the intention clear and be safe if the default changes.
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.
I added some more detailed explanation about this to the manual in #44248
@GrahamcOfBorg build givaro |
Success on x86_64-linux (full log) Attempted: givaro Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: givaro Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: givaro Partial log (click to expand)
|
For some reason github didn't notify me about your changes. No idea why, but maybe adding a comment that you think it is ready for review again would work around this github issue. @GrahamcOfBorg eval |
Success on x86_64-linux (full log) Attempted: givaro Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: givaro Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: givaro Partial log (click to expand)
|
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)