Merge pull request #877 from luukvbaal/preview-tui

cleanup fifo
jarun · Mar 14, 2021 · 3c0c595 · 3c0c595
2 parents 519ea83 + 7527125
commit 3c0c595
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/plugins/preview-tui b/plugins/preview-tui
@@ -209,7 +209,10 @@ ueberzug_refresh() {
     preview_fifo &
     wait
 }
-[ "$TERMINAL" != "kitty" ] && [ "$PREVIEW_MODE" ] && exists ueberzug && trap 'ueberzug_refresh' WINCH
+if [ "$TERMINAL" != "kitty" ] && [ "$PREVIEW_MODE" ] && exists ueberzug; then
+    trap 'ueberzug_refresh' WINCH
+    trap 'rm "$FIFO_UEBERZUG"' INT HUP EXIT
+fi
 
 preview_fifo() {
     # use cat instead of 'exec <' to avoid issues with dash shell

diff --git a/plugins/preview-tui-ext b/plugins/preview-tui-ext
@@ -272,7 +272,10 @@ ueberzug_refresh() {
     preview_fifo &
     wait
 }
-[ "$TERMINAL" != "kitty" ] && [ "$PREVIEW_MODE" ] && exists ueberzug && trap 'ueberzug_refresh' WINCH
+if [ "$TERMINAL" != "kitty" ] && [ "$PREVIEW_MODE" ] && exists ueberzug; then
+    trap 'ueberzug_refresh' WINCH
+    trap 'rm "$FIFO_UEBERZUG"' INT HUP EXIT
+fi
 
 preview_fifo() {
     # use cat instead of 'exec <' to avoid issues with dash shell