diff options
author | Robert Hensing <robert@roberthensing.nl> | 2023-01-31 15:10:12 +0100 |
---|---|---|
committer | Robert Hensing <robert@roberthensing.nl> | 2023-01-31 15:12:04 +0100 |
commit | 5cfdab8950f275a8d0370d75bcc7b7f41ce8ee19 (patch) | |
tree | d1eaa04a9407179635645fc4c6b114d866c8fab5 /maintainers/scripts | |
parent | b6fee1a4eb5d3f4fc565c1f898c9852c3b519bea (diff) |
eval-release.nix: Tolerate null
This will allow tests.pkg-config.defaultPkgConfigPackages to run on hydra without breaking the tarball job. Regarding the use of null, I'll quote 473ac96 which does lib.hydraJob. By allowing null, we allow code to avoid filterAttrs, improving laziness in real world use cases. Specifically, this strategy prevents infinite recursion errors, performance issues and possibly other errors that are unrelated to the user's code.
Diffstat (limited to 'maintainers/scripts')
-rw-r--r-- | maintainers/scripts/eval-release.nix | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/maintainers/scripts/eval-release.nix b/maintainers/scripts/eval-release.nix index bb9572cbc7956..4f0ca24650251 100644 --- a/maintainers/scripts/eval-release.nix +++ b/maintainers/scripts/eval-release.nix @@ -17,6 +17,7 @@ let if (builtins.tryEval attrs.drvPath).success then { inherit (attrs) name drvPath; } else { failed = true; } + else if attrs == null then {} else { recurseForDerivations = true; } // mapAttrs (n: v: let path' = path ++ [n]; in trace path' (recurse path' v)) attrs else { }; |