Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix autostart #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

proletarius101
Copy link
Contributor

Currently, this flatpak doesn't grant the ProtonMail Bridge the filesystem=xdg-config/autostart permission, which means it will only create an autostart entry in the shadow directory. This PR fixes the issue.

@flathubbot
Copy link
Contributor

Started test build 46371

@flathubbot
Copy link
Contributor

Build 46371 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/44608/ch.protonmail.protonmail-bridge.flatpakref

@A6GibKm
Copy link
Collaborator

A6GibKm commented Apr 27, 2021

Did you test that it works? I dont remember which hack I have in place for it to autostart so I cant test it properly.

@proletarius101
Copy link
Contributor Author

proletarius101 commented Apr 27, 2021

Did you test that it works? I dont remember which hack I have in place for it to autostart so I cant test it properly.

This could make a ~/.config/autostart/ProtonMail\ Bridge.desktop.

[Desktop Entry]
Type=Application
Name=ProtonMail Bridge
Exec="/app/lib/protonmail/bridge/proton-bridge" "--no-window"
X-GNOME-Autostart-enabled=true

However, I just realize it's not enough. It has to be flatpak run ch.protonmail.protonmail-bridge --no-window. Therefore, it must be "build from source" and we add a patch.

@rowbawts
Copy link

Any solutions for this?

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.

4 participants