better error if no disks are configured

This commit is contained in:
Jörg Thalheim 2024-08-26 18:12:28 +02:00
parent 54bf06ba8c
commit d4c64aee75

View file

@ -98,16 +98,21 @@ in
};
};
};
config = lib.mkIf (cfg.devices.disk != { }) {
system.build = (cfg.devices._scripts { inherit pkgs; checked = cfg.checkScripts; }) // {
# we keep these old outputs for compatibility
disko = builtins.trace "the .disko output is deprecated, please use .diskoScript instead" (cfg.devices._scripts { inherit pkgs; }).diskoScript;
diskoNoDeps = builtins.trace "the .diskoNoDeps output is deprecated, please use .diskoScriptNoDeps instead" (cfg.devices._scripts { inherit pkgs; }).diskoScriptNoDeps;
config = {
system.build = lib.mapAttrs (name: cfg:
if cfg.devices.disk == {} then
throw ''
Cannot provide ${name} without disks configured.
Current value of disko.devices.disk is empty.
Set up a disk configuration according to:
https://github.com/nix-community/disko/blob/master/docs/quickstart.md#step-1-choose-a-disk-configuration
''
else cfg)
(cfg.devices._scripts { inherit pkgs; checked = cfg.checkScripts; } // {
diskoImages = diskoLib.makeDiskImages {
nixosConfig = args;
};
diskoImagesScript = diskoLib.makeDiskImagesScript {
nixosConfig = args;
};
@ -121,7 +126,7 @@ in
extraSystemConfig = cfg.tests.extraConfig;
extraTestScript = cfg.tests.extraChecks;
};
};
});
# we need to specify the keys here, so we don't get an infinite recursion error