Signed-off-by: Sébastien Luttringer <[email protected]>
---
 makechrootpkg.in | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/makechrootpkg.in b/makechrootpkg.in
index 7a780df..0e2d5c0 100644
--- a/makechrootpkg.in
+++ b/makechrootpkg.in
@@ -22,6 +22,7 @@ run_namcap=false
 chrootdir=
 passeddir=
 declare -a install_pkgs
+declare -i ret=0
 
 default_copy=$USER
 [[ -n $SUDO_USER ]] && default_copy=$SUDO_USER
@@ -160,7 +161,6 @@ if [[ ! -d $copydir ]] || $clean_first; then
 fi
 
 if [[ -n "${install_pkgs[*]}" ]]; then
-       declare -i ret=0
        for install_pkg in "${install_pkgs[@]}"; do
                pkgname="${install_pkg##*/}"
                cp "$install_pkg" "$copydir/$pkgname"
@@ -277,9 +277,7 @@ cat >"$copydir/chrootbuild" <<EOF
 export HOME=/build
 
 cd /build
-sudo -u nobody makepkg $makepkg_args || touch BUILD_FAILED
-
-[[ -f BUILD_FAILED ]] && exit 1
+sudo -u nobody makepkg $makepkg_args || exit 1
 
 if $run_namcap; then
        pacman -S --needed --noconfirm namcap
@@ -313,7 +311,7 @@ if mkarchroot -r "/chrootbuild" "$copydir"; then
        done
 else
        # Just in case. We returned 1, make sure we fail
-       touch "$copydir/build/BUILD_FAILED"
+       ret=1
 fi
 
 for f in "$copydir"/srcdest/*; do
@@ -321,7 +319,6 @@ for f in "$copydir"/srcdest/*; do
        mv "$f" "$SRCDEST"
 done
 
-if [[ -e $copydir/build/BUILD_FAILED ]]; then
-       rm "$copydir/build/BUILD_FAILED"
+if (( ret != 0 )); then
        die "Build failed, check $copydir/build"
 fi
-- 
Sébastien "Seblu" Luttringer

Reply via email to