From: martin f. krafft <madduck@madduck.net>
Date: Tue, 30 Jul 2019 20:30:50 +0000 (+1200)
Subject: bgrun: notify about output during cleanup
X-Git-Url: https://git.madduck.net/etc/mutt.git/commitdiff_plain/8937bb7554dbcfad9face59b9d9b5c4368fcff6e?ds=inline

bgrun: notify about output during cleanup
---

diff --git a/.mutt/bgrun b/.mutt/bgrun
index 82e1748..d4278b2 100755
--- a/.mutt/bgrun
+++ b/.mutt/bgrun
@@ -9,7 +9,7 @@ elif [ -d "${TMPDIR}/volatile" ]; then
 fi
 export TMPDIR
 TMPDIR=$(mktemp -dp "$TMPDIR" mutt.XXXXXXXXXX)
-cleanup() { cd / && rm -rf "$TMPDIR"; }
+cleanup() { cd / && notify_output && rm -rf "$TMPDIR"; }
 trap cleanup 1 2 3 4 5 6 7 8 10 11 12 13 14 15
 
 cd "$TMPDIR"
@@ -29,6 +29,13 @@ notify() {
   fi
 }
 
+notify_output() {
+  if [ -s "$TMPDIR/output.stdout" ] || [ -s "$TMPDIR/output.stderr" ]; then
+    notify $TMPDIR/output.stdout $TMPDIR/output.stderr \
+      "Output from mutt/$SELF on $BASENAME"
+  fi
+}
+
 guess_extension() {
   python -c "import mimetypes; print(mimetypes.guess_extension('$1'))"
 }
@@ -52,13 +59,6 @@ get_file() {
   fi
 }
 
-notify_output() {
-  if [ -s "$TMPDIR/output.stdout" ] || [ -s "$TMPDIR/output.stderr" ]; then
-    notify $TMPDIR/output.stdout $TMPDIR/output.stderr \
-      "Output from mutt/$SELF on $BASENAME"
-  fi
-}
-
 case "$SELF" in
   (bgrun)
     # make a copy of the file, then launch a shell process in the background
@@ -67,7 +67,6 @@ case "$SELF" in
     FILE="$(get_file "${1:-}")"
     (
       launch_viewer "$MIMETYPE" "$FILE"
-      notify_output
       sleep 1
       cleanup
     ) &
@@ -86,7 +85,6 @@ case "$SELF" in
     # the previous three lines:
     ## cat > "$FIFO" &
     launch_viewer "$MIMETYPE" "${FIFO}"
-    notify_output
     wait
     cleanup
     ;;
@@ -96,7 +94,11 @@ case "$SELF" in
     # cases when the FIFO method doesn't work, because e.g. Firefox randomly
     # chooses it needs to read HTML files twice.
     FILE="$(get_file "${1:-}")"
-    (launch_viewer "$MIMETYPE" "${FILE}" && notify_output) &
-    (sleep 1m && cleanup) &
+    (launch_viewer "$MIMETYPE" "${FILE}") &
+    (
+      sleep 1m
+      echo Cleaning up $TMPDIR… > $TMPDIR/output.stderr
+      cleanup
+    ) &
     ;;
 esac