Skip to content

Fix: dynamic playlists with flatpak build#123

Merged
nullobsi merged 4 commits into
nullobsi:mainfrom
maxmitti:fix_flatpak_dynamic_playlists
Feb 19, 2026
Merged

Fix: dynamic playlists with flatpak build#123
nullobsi merged 4 commits into
nullobsi:mainfrom
maxmitti:fix_flatpak_dynamic_playlists

Conversation

@maxmitti

Copy link
Copy Markdown
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

lokke3 commented Dec 17, 2025

Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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

maxmitti commented Dec 22, 2025

Copy link
Copy Markdown
Contributor Author

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

maxmitti commented Jan 4, 2026

Copy link
Copy Markdown
Contributor Author

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
@nullobsi nullobsi merged commit 134c485 into nullobsi:main Feb 19, 2026
4 checks passed
@maxmitti maxmitti deleted the fix_flatpak_dynamic_playlists branch June 7, 2026 13:18
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: perl not found when using dynamic playlist feature bug: Flatpak installation cannot find perl binary on linux when generating dynamic playlists

3 participants