improved lock handling

diff --git a/.bin/offlineimap b/.bin/offlineimap
index 82428cd7526f636d0450207ca338184b1af92357..ae82806c5753b37d08ee2a94fcd47dd96cfa3395 100755 (executable)
--- a/.bin/offlineimap
+++ b/.bin/offlineimap
@@ -5,12 +5,15 @@ set -eu
 
 VAR=$HOME/.var/offlineimap
 LOCKFILE="$VAR/.cron.lock"
 
 VAR=$HOME/.var/offlineimap
 LOCKFILE="$VAR/.cron.lock"

-trap "rm -f $LOCKFILE" 0 1 2 3 4 5 6 7 8 10 11 12 13 14 15
+trapfn() { rm -f "$LOCKFILE"; trap - 0 1 2 3 4 5 6 7 8 10 11 12 13 14 15; }
+trap trapfn 1 2 3 4 5 6 7 8 10 11 12 13 14 15

 if ! lockfile -0 -r0 -l 3600 "$LOCKFILE" 2>/dev/null; then
   echo "E: offlineimap is already running." >&2
   exit 1
 fi
 
 if ! lockfile -0 -r0 -l 3600 "$LOCKFILE" 2>/dev/null; then
   echo "E: offlineimap is already running." >&2
   exit 1
 fi
 

+trap trapfn 0
+

 if ! test -t 0; then
   echo Renicing because we are not run interactively…
   renice 20 -p $$ >/dev/null
 if ! test -t 0; then
   echo Renicing because we are not run interactively…
   renice 20 -p $$ >/dev/null


@@ -18,4 +21,7 @@ if ! test -t 0; then
 fi
 
 ret=0
 fi
 
 ret=0

-/usr/bin/offlineimap "$@" || exit $?
+/usr/bin/offlineimap "$@" || ret=$?
+
+trapfn
+exit $ret