From 965cc5da574c8ee2310d0cc0a7d75363f422b4c3 Mon Sep 17 00:00:00 2001 From: John Ericson Date: Sat, 9 Dec 2017 14:27:24 -0500 Subject: top-level: Fix splicing in the presence of evaluation errors - No more `or {}`, this was misleading at best since those values wouldn't be used unless the attr they are defined from was present anyways. - `tryEval` for get outputs. This ensures that if some derivations fail, they won't take out the others. This benefited from the `or {}` before, but that was never good enough. `tryEval` is strictly better. --- pkgs/top-level/splice.nix | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'pkgs/top-level/splice.nix') diff --git a/pkgs/top-level/splice.nix b/pkgs/top-level/splice.nix index b13fa86a9958..d2c339b04df8 100644 --- a/pkgs/top-level/splice.nix +++ b/pkgs/top-level/splice.nix @@ -37,12 +37,15 @@ let inherit name; value = let defaultValue = mash.${name}; - buildValue = buildPkgs.${name} or {}; - runValue = runPkgs.${name} or {}; + buildValue = buildPkgs.${name}; + runValue = runPkgs.${name}; augmentedValue = defaultValue // (lib.optionalAttrs (buildPkgs ? ${name}) { nativeDrv = buildValue; }) // (lib.optionalAttrs (runPkgs ? ${name}) { crossDrv = runValue; }); # Get the set of outputs of a derivation + tryGetOutputs = value0: let + eval = builtins.tryEval value0; + in getOutputs (if eval.success then eval.value else {}); getOutputs = value: lib.genAttrs (value.outputs or (lib.optional (value ? out) "out")) (output: value.${output}); @@ -54,7 +57,7 @@ let # The derivation along with its outputs, which we recur # on to splice them together. else if lib.isDerivation defaultValue then augmentedValue - // splicer (getOutputs buildValue) (getOutputs runValue) + // splicer (tryGetOutputs buildValue) (getOutputs runValue) # Just recur on plain attrsets else if lib.isAttrs defaultValue then splicer buildValue runValue # Don't be fancy about non-derivations. But we could have used used -- cgit 1.4.1