-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
treewide: never assert stdenv.isLinux or stdenv.isDarwin #37029
Conversation
meta.platforms can handle this for us
I think there's a problem though that the attributes of the derivation are often evaluated before meta.platforms - e.g. the Borg reports:
|
Or hmm, maybe it's actually that |
Or maybe it's that |
Closing for now. If anyone wants to take this cause up, feel free to reuse this stuff. |
Assertions shouldn't be used in Nixpkgs for checking the platform. "meta.platforms" provides a much more robust way of handling this.
This treewide commit removes all references to 'assert stdenv.isLinux' and 'assert stdenv.isDarwin'.