Skip to content
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: meta.platforms fixes now that it tracks the host platform #37460

Merged
merged 4 commits into from Mar 20, 2018

Conversation

Ericson2314
Copy link
Member

Motivation for this change

In #34444 I made meta.platforms track the host platform instead of the build platform. Now a few packages need to be updated so that the cross jobs don't all disappear.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Otherwise obscure cross-compilations are hampered. `all` breaks all but
the initial derivation (which we can't even write yet) in an open world
setting however, so we really shouldn't have it.
Since GHC is a cross compiler, it's perfectly possible to make haskell
binaries on platforms without GHCs. `windows ++ unix` seems good enough
for now.

Also don't default `hydraPlatforms` to `platforms`. The former must be a
list of systems (strings), but the latter is a list of systems or
patterns.
@Ericson2314 Ericson2314 merged commit 0c52fe9 into NixOS:master Mar 20, 2018
@Ericson2314 Ericson2314 deleted the host-meta-platforms-fixes branch March 20, 2018 17:08
@Ericson2314
Copy link
Member Author

Ericson2314 commented Mar 20, 2018

https://hydra.nixos.org/eval/1442332?compare=1440048#tabs-removed is the only jobs still removed, all of which failed. This seemed good enough.

@Ericson2314 Ericson2314 added this to After big PR in Cross compilation Mar 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Cross compilation
After big PR
Development

Successfully merging this pull request may close these issues.

None yet

2 participants