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

Trying to fix #92 #93

Closed
wants to merge 4 commits into from
Closed

Trying to fix #92 #93

wants to merge 4 commits into from

Conversation

bermeitinger-b
Copy link
Collaborator

No description provided.

@flathubbot
Copy link
Contributor

Started test build 7510

@flathubbot
Copy link
Contributor

Build 7510 failed

@flathubbot
Copy link
Contributor

Started test build 7511

@flathubbot
Copy link
Contributor

Build 7511 failed

@flathubbot
Copy link
Contributor

Started test build 7512

@flathubbot
Copy link
Contributor

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

flatpak install --user https://dl.flathub.org/build-repo/7170/org.signal.Signal.flatpakref

@flathubbot
Copy link
Contributor

Started test build 7516

@flathubbot
Copy link
Contributor

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

flatpak install --user https://dl.flathub.org/build-repo/7174/org.signal.Signal.flatpakref

@bermeitinger-b
Copy link
Collaborator Author

This disables the electron sandbox by adding the parameter --no-sandbox to the app start. I didn't find another possibility @barthalion

@Cerberus0
Copy link
Contributor

Consider updating the app to version 1.27.2 first. The changelog says:

Fixes a startup issue present on some Linux distributions.

It may or may not fix #92.

@SISheogorath
Copy link

LOL:

signalapp/Signal-Desktop@1ca0d82

It simply forces --no-sandbox into the desktop file. That's definitely not how it's supposed to work…

@bermeitinger-b
Copy link
Collaborator Author

Apparently, they do the same thing in 1.27.2 as this pull requests suggest. So, I'm closing this in favour of the update.

@bermeitinger-b bermeitinger-b deleted the chrome-sandbox branch September 7, 2019 17:40
@bermeitinger-b bermeitinger-b mentioned this pull request Sep 7, 2019
@SISheogorath
Copy link

When I saw it correct, this PR would still be required since flatpak doesn't use the same desktop file.

@binnih
Copy link

binnih commented Sep 7, 2019

Same problem with 1.27.2

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.

5 participants