Skip to content

fix(release): mainBinaryName so Flatpak packaging finds the binary#86

Merged
johnjazzi merged 1 commit into
mainfrom
dev
Jun 10, 2026
Merged

fix(release): mainBinaryName so Flatpak packaging finds the binary#86
johnjazzi merged 1 commit into
mainfrom
dev

Conversation

@johnjazzi

Copy link
Copy Markdown
Contributor

Why

The release run for #85 failed in build-flatpak (both arches):

ar -x canonic.deb
install: cannot stat 'usr/bin/canonic': No such file or directory
Error: module canonic: Child process exited with code 1

The Tauri .deb named the executable after the Cargo bin (app), but the
Flatpak manifest, command:, .desktop and icon paths all reference
canonic. With mainBinaryName unset, Tauri fell back to the crate bin name.

Fix

Set "mainBinaryName": "canonic" in tauri.conf.json. The bundled binary,
.desktop and icons are now canonic.*, satisfying every manifest reference.
Updater asset names are productName-derived (canonic.app.tar.gz,
canonic_*_x64-setup.exe, *.AppImage), so they're unchanged and latest.json
URLs still resolve.

Verified: built .deb from the failed run contains usr/bin/app; manifest
expects usr/bin/canonic. cargo check passes; 279 tests pass.

Context

Completes the updater/release fixes whose siblings already merged via #85:

  • clippy -D warnings errors (FETCH_HEAD format!, redundant u64 cast)
  • core:window:allow-set-focus capability (titlebar setFocus)
  • verify-manifest post-release smoke test for latest.json

Merging this triggers a clean release that should publish a signed latest.json.

🤖 Generated with Claude Code

The Tauri deb named the binary after the cargo bin ('app'), but the Flatpak
manifest, command:, .desktop and icons all reference 'canonic'. flatpak-builder
failed: install: cannot stat 'usr/bin/canonic'. Setting mainBinaryName=canonic
makes the bundled binary, .desktop and icons all 'canonic.*', matching the
manifest. Updater asset names are productName-derived, so unchanged.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@johnjazzi johnjazzi merged commit e79357e into main Jun 10, 2026
5 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.

1 participant