]> git.madduck.net Git - etc/mutt.git/commitdiff

madduck's git repository

Every one of the projects in this repository is available at the canonical URL git://git.madduck.net/madduck/pub/<projectpath> — see each project's metadata for the exact URL.

All patches and comments are welcome. Please squash your changes to logical commits before using git-format-patch and git-send-email to patches@git.madduck.net. If you'd read over the Git project's submission guidelines and adhered to them, I'd be especially grateful.

SSH access, as well as push access can be individually arranged.

If you use my repositories frequently, consider adding the following snippet to ~/.gitconfig and using the third clone URL listed for each project:

[url "git://git.madduck.net/madduck/"]
  insteadOf = madduck:

bgrun: notify about output during cleanup
authormartin f. krafft <madduck@madduck.net>
Tue, 30 Jul 2019 20:30:50 +0000 (08:30 +1200)
committermartin f. krafft <madduck@madduck.net>
Tue, 30 Jul 2019 20:30:50 +0000 (08:30 +1200)
.mutt/bgrun

index 82e1748a448de9ccc91abebc53ec16084ab1ccbd..d4278b2fe35298905d1a9dbe28ba141743018ba0 100755 (executable)
@@ -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