stdenv: add pkgsCross.powernv.linux_latest to tests.cross.sanity #245970
Conversation
|
@ofborg build pkgsCross.powernv.linux_latest |
ofborg
bot
added
6.topic: cross-compilation
| @@ -133,6 +133,7 @@ let | |||
| pkgs.pkgsMusl.stdenv | |||
| pkgs.pkgsLLVM.stdenv | |||
| pkgs.pkgsStatic.bash | |||
| pkgs.pkgsCross.powernv.linux_latest | |||
There was a problem hiding this comment.
IMO the kernel is too heavy to be here because it takes a massive amount of memory to build, and the kernel is not really “cross sanity”
There was a problem hiding this comment.
Note: these expressions are not built automatically by anything, anywhere (not Hydra, not OfBorg). It's an (executable) list of reminders for people who are doing large refactorings, so they can easily check that they aren't breaking things they personally don't use often.
Shall I split sanity into sanity-small and sanity-full?
The kernel builds need to be in here somewhere. The kernel has totally bespoke and different build processes for different architectures (yuck, I know).
There was a problem hiding this comment.
When imagining sanity one thinks of sane compilers or build environments
How about a separate list for the kernels? tests.cross.kernels
ghost
requested a review
from 
ghost
closed this
Description of changes
This commit adds
pkgsCross.powernv.linux_latesttotests.cross.sanitynow that it builds again.Targeting
stagingbecause this needs #245449.Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)