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

Set SUID bit on chrome-sandbox for Debian #10580

Merged
merged 2 commits into from
Aug 16, 2019

Conversation

jryans
Copy link
Collaborator

@jryans jryans commented Aug 16, 2019

This tweaks Linux packages for Riot to SUID chrome-sandbox after install. This
is required as of Electron 5 for certain distros, such as Debian.

This change has also been provided to electron-builder upstream, so ideally
they'll include it in the future and this becomes redundant.

Fixes #10509

This tweaks Linux packages for Riot to SUID `chrome-sandbox` after install. This
is required as of Electron 5 for certain distros, such as Debian.

This change has also been provided to `electron-builder` upstream, so ideally
they'll include it in the future and this becomes redundant.

Fixes #10509
@jryans jryans requested a review from a team August 16, 2019 12:39
Copy link
Member

@dbkr dbkr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, although presumably this is duplicating the whole after-install file because the custom one replaces the standard one, so the comment means "delete this whole file" rather than, "remove this line"?

@jryans
Copy link
Collaborator Author

jryans commented Aug 16, 2019

presumably this is duplicating the whole after-install file because the custom one replaces the standard one, so the comment means "delete this whole file" rather than, "remove this line"?

Yes, precisely. I'll tweak the comment to make that more obvious.

@jryans jryans merged commit 8824f7e into develop Aug 16, 2019
@batterystaples
Copy link

This issue prevents Riot from working when it is installed. Although the fix is simple (change the permissions of the chrome-sandbox file), many users who are not technical will install it, see that it doesn't work, and then give up. It also breaks things for users upgrading from an older version. This issue is significant enough that (in my view) it would justify making a special update to the packages in the repository just to fix this issue as soon as possible.

@jryans
Copy link
Collaborator Author

jryans commented Sep 16, 2019

This issue prevents Riot from working when it is installed. Although the fix is simple (change the permissions of the chrome-sandbox file), many users who are not technical will install it, see that it doesn't work, and then give up. It also breaks things for users upgrading from an older version. This issue is significant enough that (in my view) it would justify making a special update to the packages in the repository just to fix this issue as soon as possible.

I don't understand if you are saying this change fixes such an issue (which it was meant to) or creates a new one... If an issue exists still, please open a new issue so we can investigate.

@batterystaples
Copy link

This change fixes an issue. However, the old, unfixed version is still in the official repositories, which was the point behind my comment.

@t3chguy t3chguy deleted the jryans/linux-electron-5-sandbox branch May 12, 2022 09:04
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.

FATAL Unable to start riot 1.3.2 after upgrade
3 participants