Skip to content

Fix: dynamic playlists with flatpak build#123

Open
maxmitti wants to merge 4 commits intonullobsi:mainfrom
maxmitti:fix_flatpak_dynamic_playlists
Open

Fix: dynamic playlists with flatpak build#123
maxmitti wants to merge 4 commits intonullobsi:mainfrom
maxmitti:fix_flatpak_dynamic_playlists

Conversation

@maxmitti
Copy link
Contributor

Fixes #91

Starting and stopping dynamic playlists with the flatpak build works.
However, when Cantata is quit while the dynamic playlist is running the cantata-dynamic script keeps running and can only be stopped by killing it externally.

@lokke3
Copy link

lokke3 commented Dec 17, 2025

IIRC the behavior of the dynamic playlist continuing to run after quitting Cantata has existed for a long time. Not sure if that is actually a bug.

@nullobsi
Copy link
Owner

IIRC the behavior of the dynamic playlist continuing to run after quitting Cantata has existed for a long time. Not sure if that is actually a bug.

Ideally it would start/stop with Cantata. I do not think it translates well to Flatpak (it will show a "running" indicator even when it is not running).

@nullobsi
Copy link
Owner

Regardless, I think a partial fix is as good as any. I hope that the changes will work on the Flathub build too.

@nullobsi nullobsi force-pushed the fix_flatpak_dynamic_playlists branch from 78f5957 to 0df675e Compare December 19, 2025 04:54
@maxmitti
Copy link
Contributor Author

maxmitti commented Dec 22, 2025

IIRC the behavior of the dynamic playlist continuing to run after quitting Cantata has existed for a long time. Not sure if that is actually a bug.

It seems intended, especially because there is also a systemd service file: https://github.com/nullobsi/cantata/blob/main/playlists/cantata-dynamic.service

Regardless, it is buggy with the flatpak, because a newly started instance can’t communicate with the cantata-dynamic instance from the previous instance.
I also think that stopping the script when Cantata (in flatpak) is closed is the easiest fix for that.

@maxmitti
Copy link
Contributor Author

maxmitti commented Jan 4, 2026

I had another idea: The dynamic playlist socket could be located in the XDG runtime directory instead, which should be accessible by newly started instances as well.

Nevermind, I just realized that the communication happens via mpd channels, so that is not the issue.

@maxmitti maxmitti force-pushed the fix_flatpak_dynamic_playlists branch from 0df675e to 134c485 Compare January 4, 2026 21:50
@maxmitti maxmitti marked this pull request as ready for review January 4, 2026 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: Flatpak installation cannot find perl binary on linux when generating dynamic playlists bug: perl not found when using dynamic playlist feature

3 participants