Skip to content

fix: clone ffmpeg from GitHub instead of fetching ffmpeg.org - #6

Merged
shimonewman merged 1 commit into
mainfrom
fix-ffmpeg-github-clone
Nov 5, 2026
Merged

shimonewman merged 1 commit into
mainfrom
fix-ffmpeg-github-clone

Conversation

@shimonewman

Copy link
Copy Markdown
Collaborator

What

The ffmpeg build stage now clones the release tag from the GitHub mirror with a shallow checkout instead of downloading the source tarball from ffmpeg.org. This matches how the swoole stage in this Dockerfile, and the other utopia-php Dockerfiles, already fetch their pinned sources.

Why

ffmpeg.org resets or times out connections from GitHub Actions runners. In appwrite/appwrite it failed four image builds in a row today (Recv failure: Connection reset by peer, and one Failed to connect ... after 133568 ms), and curl retries do not outlast an outage of that length. GitHub is reliable from runners. The same change landed in the Appwrite Dockerfile in appwrite/appwrite#15236.

Notes

  • git replaces curl and xz in the stage's apk list.
  • The Renovate comment already tracks FFmpeg/FFmpeg tags with the n prefix, so FFMPEG_VERSION keeps updating and the clone uses n${FFMPEG_VERSION} directly.
  • A git checkout has no generated VERSION file, so ffmpeg -version reports the tag name (n8.1.2) rather than 8.1.2. Nothing in the library parses that string; Process::exists() only checks the binary runs.

🤖 Generated with Claude Code

ffmpeg.org resets or times out connections from GitHub Actions runners,
and curl retries do not outlast it. Clone the release tag from the
GitHub mirror with a shallow checkout, the same way the swoole stage
already builds its pinned source. The Renovate rule already tracks
FFmpeg/FFmpeg tags with the n prefix, so the pin keeps updating.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@shimonewman
shimonewman merged commit c7dce37 into main Nov 5, 2026
5 checks passed
@shimonewman
shimonewman deleted the fix-ffmpeg-github-clone branch November 5, 2026 13:14
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