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
dotnetCorePackages.sdk_5_0: init at version 5.0.100-preview.8.20417.9 #97806
Conversation
Co-authored-by: Felix Tenley <felschr@pm.me>
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'm not entirely sure about this but there seems generally no reason against including preview releases in nixpkgs.
Considering that .NET 5 is in the final stages of development personally I'd approve.
Thanks. Yeah, I know it's close to GA. I've missed the 20.09 cutoff, but I figure it'll be GA in time for 21.03, and it does no harm being in unstable in the meantime. |
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.
Reviewed points
- package name fits guidelines
- package version fits guidelines
- package build on x86_64-linux
- executables tested on x86_64-linux
Comments
PR title should be updated.
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Is there someone allocated for dotnet infrastructure? |
@AndersonTorres From the git history it looks like @jonringer & @nyanloutre are the best people to review these PRs in future. |
Yea, I've been kind of following the dotnet scene... I'm using it again for work. So I might take some time to "make it nice" to use. |
also, IIRC, dotnet 5 is supposed to unify all of the various dotnet frameworks. If a stable release of dotnet5 comes out, we might want to make it the default |
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)