summaryrefslogtreecommitdiffstats
path: root/misc-scripts/make-sourceball
diff options
context:
space:
mode:
authorPierre Schmitz <pierre@archlinux.de>2010-11-21 10:07:59 +0100
committerPierre Schmitz <pierre@archlinux.de>2010-11-21 10:07:59 +0100
commite5539fc65872aa98d510a03d1e20c207ca6ed058 (patch)
tree35f6704ad2b22837995c9e20c60d59e565a2f4ac /misc-scripts/make-sourceball
parent6dc2b46cc17fcac93d3ed1758edd53222631e849 (diff)
downloaddbscripts-e5539fc65872aa98d510a03d1e20c207ca6ed058.tar.gz
dbscripts-e5539fc65872aa98d510a03d1e20c207ca6ed058.tar.xz
sourceballs: fix error handling
Diffstat (limited to 'misc-scripts/make-sourceball')
-rwxr-xr-xmisc-scripts/make-sourceball29
1 files changed, 24 insertions, 5 deletions
diff --git a/misc-scripts/make-sourceball b/misc-scripts/make-sourceball
index da51c93..bdf5f59 100755
--- a/misc-scripts/make-sourceball
+++ b/misc-scripts/make-sourceball
@@ -25,8 +25,20 @@ create_srcpackage() {
if [ -d "$1" ]; then
pushd "$1" >/dev/null
pkgver=$(. PKGBUILD; echo ${pkgver})
+ if [ $? -ne 0 ]; then
+ error "Failed to read pkgver from $1"
+ return 1
+ fi
pkgrel=$(. PKGBUILD; echo ${pkgrel})
+ if [ $? -ne 0 ]; then
+ error "Failed to read pkgrel from $1"
+ return 1
+ fi
license=($(. PKGBUILD; echo ${license[@]}))
+ if [ $? -ne 0 ]; then
+ error "Failed to read license from $1"
+ return 1
+ fi
if ! [ $FORCE == 1 ] && ! chk_license ${license[@]} ; then
# Removed so as not to clutter failed.txt
#warning "$packagename license (${license[@]}) does not require source tarballs"
@@ -35,9 +47,11 @@ create_srcpackage() {
msg "Creating source tarball for $packagename-$pkgver-$pkgrel"
fi
- local logfile="${LOGDIR}/sourceballs/$packagename.gz"
- if ! /usr/bin/makepkg --allsource --ignorearch 2>&1 | gzip >"$logfile"; then
- die "Failed to download source for $packagename-$pkgver-$pkgrel ($reponame-$arch)"
+ local logfile="${LOGDIR}/sourceballs/$packagename"
+ if ! /usr/bin/makepkg --allsource --ignorearch >"$logfile" 2>&1; then
+ error "Failed to download source for $packagename-$pkgver-$pkgrel ($reponame-$arch)"
+ popd >/dev/null
+ return 1
fi
/bin/rm -f "$logfile"
@@ -55,10 +69,15 @@ set_umask
[ ! -d "${LOGDIR}/sourceballs" ] && mkdir -p "${LOGDIR}/sourceballs"
cd "$WORKDIR"
+failed=0
+
if /usr/bin/svn export -q "$SVNREPO/$packagename" $packagename; then
- create_srcpackage "$packagename/repos/$reponame-$arch"
+ create_srcpackage "$packagename/repos/$reponame-$arch" || failed=1
else
- die "Package '$packagename' does not exist in repo '$reponame-$arch'"
+ error "Package '$packagename' does not exist in repo '$reponame-$arch'"
+ failed=1
fi
script_unlock
+
+exit $failed \ No newline at end of file