mirror of
https://github.com/NixOS/nix.git
synced 2025-11-09 03:56:01 +01:00
169033001d
Instead of passing them around separately, or doing finicky logic in a try-catch block to recover them, just make `BuildError` always contain a status, and make it the thrower's responsibility to set it. This is much more simple and explicit. Once that change is done, split the `done` functions of `DerivationGoal` and `DerivationBuildingGoal` into separate success and failure functions, which ends up being easier to understand and hardly any duplication. Also, change the handling of failures in resolved cases to use `BuildResult::DependencyFailed` and a new message. This is because the underlying derivation will also get its message printed --- which is good, because in general the resolved derivation is not unique. One dyn drv test had to be updated, but CA (and dyn drv) is experimental, so I do not mind. Finally, delete `SubstError` because it is unused. |
||
|---|---|---|
| .. | ||
| build-built-drv.sh | ||
| common.sh | ||
| config.nix | ||
| config.nix.in | ||
| dep-built-drv-2.sh | ||
| dep-built-drv.sh | ||
| eval-outputOf.sh | ||
| failing-outer.sh | ||
| meson.build | ||
| non-trivial.nix | ||
| old-daemon-error-hack.nix | ||
| old-daemon-error-hack.sh | ||
| recursive-mod-json.nix | ||
| recursive-mod-json.sh | ||
| text-hashed-output.nix | ||
| text-hashed-output.sh | ||