Skip to content

feat(appimage): If necessary, start with --no-sandbox#9590

Open
sabonerune wants to merge 7 commits intoelectron-userland:masterfrom
sabonerune:feat/appimage-auto-no-sandbox
Open

feat(appimage): If necessary, start with --no-sandbox#9590
sabonerune wants to merge 7 commits intoelectron-userland:masterfrom
sabonerune:feat/appimage-auto-no-sandbox

Conversation

@sabonerune
Copy link
Contributor

fix #8440

The method in #4496 has the following drawbacks:

This solves these problems by automatically detecting when the user namespace is unavailable and adding --no-sandbox at runtime.

@changeset-bot
Copy link

changeset-bot bot commented Feb 13, 2026

🦋 Changeset detected

Latest commit: d3480ac

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
app-builder-lib Major
dmg-builder Major
electron-builder-squirrel-windows Major
electron-builder Major
electron-forge-maker-appimage Major
electron-forge-maker-nsis-web Major
electron-forge-maker-nsis Major
electron-forge-maker-snap Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@absidue
Copy link

absidue commented Feb 15, 2026

I would suggest changing the pull request title and changelog entry, as it sounds like you are adding the --no-sandbox flag in more cases, but the goal of this pull request is to add that flag in less cases. Something along the lines of: "feat: Only start AppImage with --no-sandbox when necessary" would be more clear.

super("appImage")

this.desktopEntry = new Lazy<string>(() => {
const args = this.options.executableArgs?.join(" ") || "--no-sandbox"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a breaking change for appimage toolset 0.0.0 (legacy) as it's AppRun script is hardcoded in upstream dependency app-builder-bin

@sabonerune sabonerune requested a review from mmaietta February 15, 2026 22:52
@sabonerune sabonerune force-pushed the feat/appimage-auto-no-sandbox branch from b752196 to 515c227 Compare February 16, 2026 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

How to run and/or install AppImage on Ubuntu

3 participants