ci: add github action to evaluate nixos options #360876
Conversation
github-actions
bot
added
6.topic: policy discussion
6.topic: continuous integration
| -I ./. \ | ||
| --option restrict-eval true \ | ||
| --option allow-import-from-derivation false \ | ||
| -A options |
There was a problem hiding this comment.
Here's what that builds:
Line 150 in df0a5e3
I do wish that line were hoisted here, rather than being referred to as "option". It's more clear that we're building the NixOS manual's JSON representation as a proxy for loading all the modules and evaluating them.
Known things it misses: evaluating all the expressions that trigger when the module's enable option is set, for instance.
There was a problem hiding this comment.
NixOS manual's JSON representation as a proxy
Isn't that practically the same as building the nixos manual? We already have a gh action for it I believe.
There was a problem hiding this comment.
@JohnRTitor there is one difference I believe. We could have optionsJSON also in staging because it doesn't require stdenv, whereas the manual might causes long rebuilds.
There was a problem hiding this comment.
Ok, but can we also build the aarch64-linux ones?
There was a problem hiding this comment.
Unsure if this is possible on x86_64. Does this make a big difference?
Things done
Unlike the manual we should be able to build this in staging as well.
Mic92#43
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.