mirror of
https://github.com/nix-community/home-manager
synced 2024-11-22 20:53:14 +00:00
home-manager: exit with an error on build failure
Because 'set -e' has no effect on commands that run in an if condition, the script was always exiting with no error when 'doBuild' failed. As a bonus, $wrkdir is now always removed after building.
This commit is contained in:
parent
ffbc7e723d
commit
a9d9fb5d75
1 changed files with 5 additions and 3 deletions
|
@ -91,11 +91,13 @@ function doSwitch() {
|
|||
local wrkdir
|
||||
wrkdir="$(mktemp -d)"
|
||||
|
||||
if doBuild "$wrkdir/generation" ; then
|
||||
"$wrkdir/generation/activate"
|
||||
fi
|
||||
local exitCode=0
|
||||
doBuild "$wrkdir/generation" && "$wrkdir/generation/activate" || exitCode=1
|
||||
|
||||
# Because the previous command never fails, the script keeps running and
|
||||
# $wrkdir is always removed.
|
||||
rm -r "$wrkdir"
|
||||
return $exitCode
|
||||
}
|
||||
|
||||
function doListGens() {
|
||||
|
|
Loading…
Reference in a new issue