search

flathub / com.github.wwmm.pulseeffects

https://flathub.org/apps/details/com.github.wwmm.pulseeffects
11 stars 6 forks source link

Autostart doesn't work #18

Closed Bleuzen closed 2 years ago

Bleuzen commented 6 years ago

PulseEffects has this nice option to "Start Service at Login". But when I toggle it on with the flatpak version, it creates the autostart file in ~/.var/app/com.github.wwmm.pulseeffects/config/autostart instead of ~/.config/autostart

Flatpak should have access to the real autostart directory to create the file there instead of in ~/.var.

Also within the autostart file, Exec=pulseeffects --gapplication-service has to be replaced with Exec=com.github.wwmm.pulseeffects --gapplication-service

TingPing commented 6 years ago

Yea it can't work, you'll just have to manually add it for now.

There is discussion about Flatpak exposing this but no work has been done on it.

wwmm commented 6 years ago

The Exec=com.github.wwmm.pulseeffects --gapplication-service will require changes in PE sources. At this moment the exec field is hardcoded. It would be necessary to patch PE sources before building

AsavarTzeth commented 6 years ago

I don't see a point in trying to workaround this issue right now. Instead we need a proper autostart portal. See issue: https://github.com/flatpak/flatpak/issues/118

Also read the comments in that issue as for why I cannot use --filesystem=xdg-config/autostart to grant access to the app itself.

For now it would probably be best to have a build time option for enabling/disabling this (and hiding it from the gui).

One might also document the limitation in the help files, and/or wiki and suggest how it can be done manually.

iQmam0n commented 6 years ago

I was able to start with the following change. ~/.config/autostart/pulseeffects-service.desktop

from: Exec=pulseeffects --gapplication-service to: Exec=/usr/bin/flatpak run --branch=stable --arch=x86_64 --command=pulseeffects com.github.wwmm.pulseeffects --gapplication-service

AsavarTzeth commented 6 years ago

@iQmamOn Yes that is how you add it manually, for now.

vchernin commented 2 years ago

This is fixed in EasyEffects, successor to PulseEffects. I have no plans to try to backport the fix here, I think solving https://github.com/flatpak/xdg-desktop-portal/issues/579 is a better use of time as that means the background portal could be used on the host.

1player commented 2 years ago

This can be closed, I've been using EasyEffects with autostart for months and it's never failed me.