1
0
Fork 0
mirror of https://github.com/nix-community/home-manager synced 2024-11-24 03:59:46 +01: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.

(cherry picked from commit a9d9fb5d75)
This commit is contained in:
Cornelius Mika 2017-08-17 10:16:25 +02:00 committed by Robert Helgesson
parent 5f2a6d90bd
commit 463d6db90b
No known key found for this signature in database
GPG key ID: C3DB11069E65DC86

View file

@ -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() {