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: Dont use obsolete platform aliases #45820
treewide: Dont use obsolete platform aliases #45820
Conversation
It is deprecated and will be removed after 18.09.
…stuff Want to get this out of here for 18.09, so it can be deprecated thereafter.
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.
It's a pity always having to add something like hostPlatform
when you don't know/use/care about cross-compilation, but I think it is a small cost for having such feature.
@FRidh yeah I'm going to try to make most of these ones be |
I definitely agree in having
suggests that one might be able to use
But as far as I know the result of that wouldn't do the right thing. ( |
But for the rest... I agree with this:
Thus I would prefer things in this order: |
I'd counter that inspecting |
No, sometimes you really need an specific platform to choose from (for instance, when downloading for some prebuilt binaries for different architectures). |
fixup from NixOS#45820 /cc @Ericson2314
Yes, this breaks stuff. Have you noticed the windows builds on Hydra? Before:
After:
|
I also wonder why Borg detected some aarch64 rebuilds. |
Motivation for this change
Should be the less controversial part of #45717. Doesn't formally deprecate anything, but rewrites usages of platform-related things I'd like to deprecate, analogous for #44423 for
configureFlags
.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)