Skip to content

Commit 20a487e

Browse files
TigerShark900claude
andcommitted
fix: detect user-level flatpak mpv install (io.mpv.Mpv, not io.mpv/Mpv)
Fedora Workstation ships no mpv package by default (needs RPM Fusion, off by default), so most Fedora users install mpv via Flatpak/GNOME Software. The Linux player-detection path checked for a directory at $HOME/.local/share/flatpak/app/io.mpv/Mpv/ (slash), but the real Flatpak app-id directory uses dots throughout: io.mpv.Mpv. That check could never match a real user-level flatpak install of mpv, so ani-cli would die with "No player found. Looked for mpv and vlc" even when mpv was correctly installed. Confirmed against live Flathub metadata (flatpak remote-info io.mpv.Mpv -> Ref: app/io.mpv.Mpv/x86_64/stable) and by simulating dep_ch_failover against both path forms. The system-wide path (/var/lib/flatpak/app/io.mpv.Mpv/, added in b0993d4) was already correct and is unchanged. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
1 parent b0993d4 commit 20a487e

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

ani-cli

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#!/bin/sh
22

3-
version_number="5.0.1"
3+
version_number="5.0.2"
44

55
# UI
66

@@ -407,7 +407,7 @@ case "$(uname -a | cut -d " " -f 1,3-)" in
407407
player_function="${ANI_CLI_PLAYER:-iSH}"
408408
curl_exe=$(dep_ch_failover "curl_safari260_ios,$curl_exe")
409409
;; # iOS (iSH)
410-
*) player_function="${ANI_CLI_PLAYER:-$(dep_ch_failover "mpv,$HOME/.local/share/flatpak/app/io.mpv/Mpv/,/var/lib/flatpak/app/io.mpv.Mpv/,vlc")}" || die 'No player found. Looked for mpv and vlc' ;; # Linux OS
410+
*) player_function="${ANI_CLI_PLAYER:-$(dep_ch_failover "mpv,$HOME/.local/share/flatpak/app/io.mpv.Mpv/,/var/lib/flatpak/app/io.mpv.Mpv/,vlc")}" || die 'No player found. Looked for mpv and vlc' ;; # Linux OS
411411
esac
412412

413413
player_extra_flags="${ANI_CLI_PLAYER_FLAGS:-""}"

0 commit comments

Comments
 (0)