Skip to content

feat: bundle FFmpeg + fix bundle.active (make release publish installers) - #14

Merged
slaveofcode merged 1 commit into
developfrom
feat/bundle-ffmpeg
Jul 25, 2026
Merged

feat: bundle FFmpeg + fix bundle.active (make release publish installers)#14
slaveofcode merged 1 commit into
developfrom
feat/bundle-ffmpeg

Conversation

@slaveofcode

Copy link
Copy Markdown
Owner

What this does

  1. Fixes the releasebundle.active was false, so tauri-action reported "No artifacts were found" even though it built and signed the app. Set to true.
  2. Bundles FFmpeg via externalBin — CI (and npm run download:ffmpeg locally) fetch a static ffmpeg per target into bin/ffmpeg-<triple> before build; nothing committed to git.

Verified locally

tauri build produced GoodWebTools.app with ffmpeg bundled at Contents/MacOS/ffmpeg, and the bundled binary runs (ffmpeg 6.0). (The local .dmg step hit a known macOS bundle_dmg.sh quirk; CI builds DMGs fine.)

Note

ffmpeg-static ships a GPL FFmpeg build — bundling it makes the distributed app GPL-encumbered (fine while GoodWebTools is open source). Documented in RELEASING-DESKTOP.md.
🤖 Generated with Claude Code

Recordings needed a system ffmpeg. Now a static ffmpeg binary is bundled inside
the app via Tauri externalBin ("bin/ffmpeg"), so there's no external dependency —
without committing ~45MB binaries to git.

- release.yml downloads the correct ffmpeg per target (eugeneware/ffmpeg-static)
  into src-tauri/bin/ffmpeg-<triple> before tauri build.
- download:ffmpeg script now actually downloads the host binary for local builds.
- tauri.conf.json: externalBin ["bin/ffmpeg"]; also flipped bundle.active to true
  (it was false, which would produce a release with NO installers).
- ffmpeg.rs already resolves bin/ffmpeg next to the executable — where Tauri
  places the sidecar — so no Rust change; system ffmpeg stays the dev fallback.
- Docs updated (RELEASING-DESKTOP.md) incl. the GPL-build license note.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013dqouzFP8vy9jaKhTDFj5H
@slaveofcode
slaveofcode merged commit f0f05fa into develop Jul 25, 2026
0 of 3 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