Browse Source

Do not leave cache mess

Alois Mahdal 7 years ago
parent
commit
e5a5eaa29d
1 changed files with 5 additions and 4 deletions
  1. 5
    4
      dotfiles/i3/bin/cstatus

+ 5
- 4
dotfiles/i3/bin/cstatus View File

@@ -33,7 +33,9 @@ mktext() {
33 33
     local value
34 34
     local key
35 35
     local color_attr
36
-    qdbus org.mpris.clementine /Player org.freedesktop.MediaPlayer.GetMetadata >"$CACHE"
36
+    local cache
37
+    cache=$(mktemp -t cstatus.GetMetadata.XXXXXXXX)
38
+    qdbus org.mpris.clementine /Player org.freedesktop.MediaPlayer.GetMetadata >"$cache"
37 39
     case $(qdbus org.mpris.clementine /org/mpris/MediaPlayer2 org.mpris.MediaPlayer2.Player.PlaybackStatus) in
38 40
         Stopped)    sigil=""; sigilcolor= ;;
39 41
         Playing)    sigil=""; sigilcolor=green ;;
@@ -51,15 +53,14 @@ mktext() {
51 53
             tracknumber)  tracknumber=$value ;;
52 54
             title)        title=$value ;;
53 55
         esac
54
-    done <"$CACHE"
56
+    done <"$cache"
55 57
     printf "%s [%d %s]: %02d %s (%s) <span%s>%s</span>" \
56 58
         "$artist" "$year" "$album" "$tracknumber" "$title" \
57 59
         "$(getpos)" \
58 60
         "$color_attr" "$sigil"
61
+    rm "$cache"
59 62
 }
60 63
 
61
-CACHE=$(mktemp -t cstatus.GetMetadata.XXXXXXXX)
62
-
63 64
 main() {
64 65
     local head=true
65 66
     echo '{"version":1}'