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
dotnet-sdk: add darwin support, set myself as maintainer #68709
Conversation
Signed-off-by: Jamie Winsor <jamie@onemoregame.com>
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.
This new expression has a lot of duplication with the default variant, conditionalizing the sources and patchelf calls would be better.
eg.
buildPhase = ''
runHook preBuild
${stdenv.lib.optionalString stdenv.isLinux ''
patchelf --set-interpreter "${stdenv.cc.bintools.dynamicLinker}" ./dotnet
patchelf --set-rpath "${rpath}" ./dotnet
find -type f -name "*.so" -exec patchelf --set-rpath '$ORIGIN:${rpath}' {} \;
''}
echo -n "dotnet-sdk version: "
./dotnet --version
runHook postBuild
'';
-change /System/Library/Frameworks/Security.framework/Versions/A/Security \ | ||
${Security}/Library/Frameworks/Security.framework/Security \ | ||
-change /System/Library/Frameworks/GSS.framework/Versions/A/GSS \ | ||
${GSS}/Library/Frameworks/GSS.framework/GSS {} \; |
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.
Frameworks and things like libc are impure on darwin so rewriting references doesn't really do anything. It's fine to keep the system references since this is a precompiled binary anyway.
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.
Ah, thank you so much for the heads up there. It wasn't clear to me that the Darwin implementation wasn't impure. ❤️
This commit adds support for the dotnet core dotnet-sdk to the Darwin platform. This is my first commit to the nixpkgs repository so it also includes a modification to the maintainers list to include an entry for myself.
Apologies if I've gotten any of the contributing guidelines wrong, I did read them, I promise!
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)