Check mkarchroot's return code

We weren't doing this before, and after build steps were
being run even if the build failed or if the user cancelled
the build

Signed-off-by: Aaron Griffin <aaronmgriffin@gmail.com>

1 files changed, 21 insertions, 18 deletions

diff --git a/makechrootpkg b/makechrootpkg
index 35ca076..209f571 100755
--- a/makechrootpkg
+++ b/makechrootpkg
@@ -218,25 +218,28 @@ EOF
 ) > "$uniondir/chrootbuild"
 chmod +x "$uniondir/chrootbuild"
 
-mkarchroot -r "/chrootbuild" "$uniondir"
-
-source ${WORKDIR}/PKGBUILD
-
-if [ -n "$add_to_db" ]; then
-    [ -d "${chrootdir}/union/repo" ] || mkdir -p "${chrootdir}/union/repo"
-    pushd "${chrootdir}/union/repo" >/dev/null
-    cp ${chrootdir}/union/pkgdest/${pkgname}-${pkgver}-${pkgrel}-*.pkg.tar.gz .
-    repo-add repo.db.tar.gz *.pkg.tar.gz
-    popd >/dev/null
-fi
+if mkarchroot -r "/chrootbuild" "$uniondir"; then
+    source ${WORKDIR}/PKGBUILD
+
+    if [ -n "$add_to_db" ]; then
+        [ -d "${chrootdir}/union/repo" ] || mkdir -p "${chrootdir}/union/repo"
+        pushd "${chrootdir}/union/repo" >/dev/null
+        cp ${chrootdir}/union/pkgdest/${pkgname}-${pkgver}-${pkgrel}-*.pkg.tar.gz .
+        repo-add repo.db.tar.gz *.pkg.tar.gz
+        popd >/dev/null
+    fi
 
-if [ -z "$(mount | grep ${chrootdir}/union/pkgdest)" ]; then
-    echo "Moving completed package file to ${WORKDIR}"
-    mv ${chrootdir}/union/pkgdest/${pkgname}-${pkgver}-${pkgrel}-*.pkg.tar.gz ${WORKDIR}
-fi
-if [ -z "$(mount | grep ${chrootdir}/union/srcdest)" ]; then
-    echo "Moving downloaded source files to ${WORKDIR}"
-    mv ${chrootdir}/union/srcdest/* ${WORKDIR}
+    if [ -z "$(mount | grep ${chrootdir}/union/pkgdest)" ]; then
+        echo "Moving completed package file to ${WORKDIR}"
+        mv ${chrootdir}/union/pkgdest/${pkgname}-${pkgver}-${pkgrel}-*.pkg.tar.gz ${WORKDIR}
+    fi
+    if [ -z "$(mount | grep ${chrootdir}/union/srcdest)" ]; then
+        echo "Moving downloaded source files to ${WORKDIR}"
+        mv ${chrootdir}/union/srcdest/* ${WORKDIR}
+    fi
+else
+    #just in case. We returned 1, make sure we fail
+    touch ${chrootdir}/rw/build/BUILD_FAILED
 fi
 
 if [ -e ${chrootdir}/rw/build/BUILD_FAILED ]; then