Skip to content

fix: repack AppImages with correct relative .DirIcon symlink#203

Merged
careck merged 2 commits into
masterfrom
worktree-fix+appimage-diricon-symlink
May 25, 2026
Merged

fix: repack AppImages with correct relative .DirIcon symlink#203
careck merged 2 commits into
masterfrom
worktree-fix+appimage-diricon-symlink

Conversation

@mattbell87
Copy link
Copy Markdown
Member

Summary

  • Adds a post-build step to .github/workflows/release.yml that extracts each Linux AppImage, replaces the broken absolute .DirIcon symlink with a relative one, repacks with appimagetool, and re-uploads to the release
  • Renames the build step from "Build and publish release" to "Build release" for clarity

Background

This is a workaround for tauri-apps/tauri#15110tauri-bundler bakes an absolute path (pointing to the CI runner's build directory) into the .DirIcon symlink inside the AppImage squashfs. Tools like app-manager follow this symlink to extract the app icon during installation and fail because the path doesn't exist on the user's machine.

Tracked in: #202

Test plan

  • Trigger a release build and verify the Linux AppImages install successfully via app-manager install
  • Verify .DirIcon inside the repacked AppImage is a relative symlink (Krillnotes.png)

Notes

Once tauri-apps/tauri#15141 or #15404 is merged upstream, this workaround will be safe to keep — it will simply overwrite an already-correct symlink with the same value.

🤖 Generated with Claude Code

mattbell87 and others added 2 commits May 18, 2026 17:05
Workaround for tauri-apps/tauri#15110 where tauri-bundler bakes an
absolute symlink for .DirIcon into the AppImage squashfs, causing
app-manager and similar tools to fail installation.

After tauri-action builds and uploads the AppImages, this step extracts
each one, replaces .DirIcon with a relative symlink, repacks with
appimagetool, and re-uploads the fixed file to the release.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mattbell87 mattbell87 requested a review from careck May 19, 2026 01:25
Copy link
Copy Markdown
Member

@careck careck left a comment

Choose a reason for hiding this comment

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

I'm not comfortable with the changes to the files which aren't related to the release workflow. Why are these included in this PR?

@mattbell87
Copy link
Copy Markdown
Member Author

I'm not comfortable with the changes to the files which aren't related to the release workflow. Why are these included in this PR?

those were to do with the pipeline not passing, I can revert if you prefer

@careck
Copy link
Copy Markdown
Member

careck commented May 25, 2026

I'm not comfortable with the changes to the files which aren't related to the release workflow. Why are these included in this PR?

those were to do with the pipeline not passing, I can revert if you prefer

Ah, sorry, it's just a formatting change, all good! I'll merge

@careck careck merged commit 67a7f74 into master May 25, 2026
2 checks passed
@careck careck deleted the worktree-fix+appimage-diricon-symlink branch May 25, 2026 01:51
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.

2 participants