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

chore: remove unnecessary debian 10 appimage fix #684

Merged
merged 1 commit into from
Dec 5, 2021

Conversation

csett86
Copy link
Member

@csett86 csett86 commented Dec 4, 2021

Debian 11 (bullseye) changed the default for kernel.unprivileged_userns_clone to 1 (see https://www.debian.org/releases/bullseye/amd64/release-notes/ch-information.en.html#linux-user-namespaces).
Thus this workaround is no longer required, remove it.

This at the same time fixes the default command line entry for the deb package.

Closes: #628

Debian 11 (bullseye) changed the default for kernel.unprivileged_userns_clone to 1,
thus this workaround is no longer required.

This at the same time fixes the default command line entry for the deb
package.

Closes: #628
@csett86 csett86 linked an issue Dec 4, 2021 that may be closed by this pull request
@csett86
Copy link
Member Author

csett86 commented Dec 4, 2021

I tested this on Debian 11 and Fedora 35, all good. Ubuntu always had kernel.unprivileged_userns_clone to 1, so they were never affected in the first place and thus dont need this as well. Thus I suggest to drop this (and with that drop support for running the AppImage on Debian 10).

@csett86 csett86 merged commit f0d020a into master Dec 5, 2021
@csett86 csett86 deleted the remove-debian-sandbox-fix branch December 5, 2021 08:09
d3473r pushed a commit to d3473r/jitsi-meet-electron that referenced this pull request Sep 1, 2023
Debian 11 (bullseye) changed the default for kernel.unprivileged_userns_clone to 1,
thus this workaround is no longer required.

This at the same time fixes the default command line entry for the deb
package.

Closes: jitsi#628
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.

Crash with Wayland native execution on 2021.12.2 Debian Package Issue
2 participants