Skip to content

fix(linux): remove libfuse2 dependency from AppImages - #1070

Merged
EtienneLescot merged 1 commit into
getopenscreen:mainfrom
satwiksharma01:fix/appimage-static-runtime
Oct 10, 2026
Merged

EtienneLescot merged 1 commit into
getopenscreen:mainfrom
satwiksharma01:fix/appimage-static-runtime

Conversation

@satwiksharma01

@satwiksharma01 satwiksharma01 commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Linux AppImages use electron-builder's legacy runtime by default, which requires libfuse2 on the host. Select the supported appimage toolset 1.0.3 in electron-builder.json5 to use its statically linked runtime.

The existing electron-builder 26.15.3 dependency supports this option. Its static-runtime packaging path still appends the embedded blockmap and generates update metadata.

Related issue

Fixes #736

Type of change

  • Bug fix

Release impact

  • Patch

Desktop impact

  • Linux
  • Installer / packaging

Screenshots / video

No visual change.

Testing

  • Validated the configuration with the installed electron-builder 26.15.3 schema.
  • Both application and test TypeScript checks passed with Node 22.22.1.
  • Repository lint passed with 26 existing warnings; documentation checks passed.
  • Full unit suite passed: 307 files, 4,298 tests passed, 5 skipped.
  • Verified the official toolset archive's SHA256 against the checksum pinned by electron-builder.
  • Verified that its x64 runtime is static PIE with no DT_NEEDED libraries.
  • Built a disposable zstd AppImage with that runtime and the toolset's mksquashfs. It mounted through FUSE and launched successfully on WSL Ubuntu 26.04 with no libfuse2 or libfuse2t64 installed.

The runtime smoke test uses a small shell application; a complete packaged OpenScreen desktop launch has not been tested. The supported toolset also uses zstd for normal compression and retains the launcher's user-namespace sandbox fallback.

Summary by CodeRabbit

  • Chores
    • No user-facing changes are included in this update. App features, behavior, and workflows remain unchanged. The update is limited to release packaging and does not affect how the app looks or works. No new capabilities or changes to existing functionality are included.

@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 13e01f2a-bff1-4fab-9a4f-e90ef3679196

📥 Commits

Reviewing files that changed from the base of the PR and between 870196c and 8c55f57.


📒 Files selected for processing (1)
  • electron-builder.json5

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 3 remain after this review.



📝 Walkthrough

Walkthrough

The build configuration pins the AppImage toolset to version 1.0.3.

Changes

AppImage Toolset

Layer / File(s) Summary
Pin the AppImage toolset version
electron-builder.json5
The configuration sets the AppImage toolset version to 1.0.3.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Bug fix


Merge Risk: ⚪ Minimal · up to 8c55f

The release workflow selects the pinned AppImage toolset, and no actionable merge-blocking risk remains. The packaged runtime artifact was not independently verified in this inspection.

Pre-merge checks | Passed 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check Passed The title clearly and concisely describes the main change: removing the libfuse2 dependency from Linux AppImages.
Description check Passed The description follows the repository template. It includes the summary, issue reference, change type, release impact, platform impact, visual-change note, and detailed testing results.
Linked Issues check Passed Issue #736 requires removing the AppImage host dependency on libfuse2. The PR adds toolsets.appimage: "1.0.3" in electron-builder.json5. The reported validation confirms that the selected x64 ru…
Out of Scope Changes check Passed The whole-PR diff contains only the appimage toolset configuration and its explanatory comment in electron-builder.json5. This change directly supports issue #736 and does not show unrelated imple…
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@EtienneLescot
EtienneLescot marked this pull request as ready for review October 10, 2026 10:39
@EtienneLescot
EtienneLescot self-requested a review as a code owner October 10, 2026 10:39
@EtienneLescot

Copy link
Copy Markdown
Collaborator

Marking ready: we're taking the static runtime (no libfuse2). The update-info hook from #1072 targets .upd_info in both runtimes and passed on a dispatched build of main.

@EtienneLescot EtienneLescot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed: static AppImage runtime (toolset 1.0.3), no libfuse2 on the host; update info still embedded. Thanks @satwiksharma01!

@EtienneLescot
EtienneLescot merged commit b87c66c into getopenscreen:main Oct 10, 2026
22 checks passed
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.

AppImage: Remove libfuse2 dependency

2 participants