1
0
Fork 0
mirror of https://github.com/hercules-ci/flake-parts.git synced 2024-12-14 11:47:31 +00:00

Memoize perSystem in allSystems, require devShell

This commit is contained in:
Robert Hensing 2021-11-22 22:01:38 +01:00
parent d5703798a8
commit 1b82c7f771
6 changed files with 20 additions and 18 deletions

View file

@ -2,7 +2,6 @@
let
inherit (lib)
filterAttrs
genAttrs
mapAttrs
mkOption
optionalAttrs
@ -30,7 +29,7 @@ in
(k: v: v.checks)
(filterAttrs
(k: v: v.checks != null)
(genAttrs config.systems config.perSystem)
config.allSystems
);
perInput = system: flake:

View file

@ -2,7 +2,6 @@
let
inherit (lib)
filterAttrs
genAttrs
mapAttrs
mkOption
optionalAttrs
@ -25,13 +24,7 @@ in
};
};
config = {
flake.devShell =
mapAttrs
(k: v: v.devShell)
(filterAttrs
(k: v: v.devShell != null)
(genAttrs config.systems config.perSystem)
);
flake.devShell = mapAttrs (k: v: v.devShell) config.allSystems;
perInput = system: flake:
optionalAttrs (flake?devShell.${system}) {
@ -42,8 +35,8 @@ in
_file = ./devShell.nix;
options = {
devShell = mkOption {
type = types.nullOr types.package;
default = null;
type = types.package;
default = throw "The default devShell was not configured for system ${system}. Please set it, or if you don't want to use the devShell attribute, set flake.devShell = lib.mkForce {};";
description = ''
A derivation that nix develop bases its environment on.
'';

View file

@ -2,7 +2,6 @@
let
inherit (lib)
filterAttrs
genAttrs
mapAttrs
mkOption
optionalAttrs

View file

@ -2,7 +2,6 @@
let
inherit (lib)
filterAttrs
genAttrs
mapAttrs
mkOption
optionalAttrs
@ -30,7 +29,7 @@ in
(k: v: v.legacyPackages)
(filterAttrs
(k: v: v.legacyPackages != null)
(genAttrs config.systems config.perSystem)
config.allSystems
);
perInput = system: flake:

View file

@ -2,7 +2,6 @@
let
inherit (lib)
filterAttrs
genAttrs
mapAttrs
mkOption
optionalAttrs
@ -31,7 +30,7 @@ in
(k: v: v.packages)
(filterAttrs
(k: v: v.packages != null)
(genAttrs config.systems config.perSystem)
config.allSystems
);
perInput = system: flake:

View file

@ -1,6 +1,7 @@
{ config, lib, self, ... }:
let
inherit (lib)
genAttrs
mapAttrs
mkOption
types
@ -36,7 +37,19 @@ in
shorthandOnlyDefinesConfig = false;
});
};
allSystems = mkOption {
type = types.lazyAttrsOf types.unspecified;
description = "The system-specific config for each of systems.";
internal = true;
};
};
config = {
perSystem = system: { _module.args.system = system; };
allSystems = genAttrs config.systems config.perSystem;
# TODO: Sub-optimal error message. Get Nix to support a memoization primop, or get Nix Flakes to support systems properly or get Nix Flakes to add a name to flakes.
_module.args.getSystem = system: config.allSystems.${system} or builtins.trace "using non-memoized system ${system}" config.perSystem system;
};
config.perSystem = system: { _module.args.system = system; };
}