Skip to content

Use pinned ffmpeg version in CI - #3276

Merged
AlexEMG merged 3 commits into
mainfrom
cy/stable-ci-ffmpeg
Apr 10, 2026
Merged

Use pinned ffmpeg version in CI#3276
AlexEMG merged 3 commits into
mainfrom
cy/stable-ci-ffmpeg

Conversation

@C-Achard

@C-Achard C-Achard commented Apr 8, 2026

Copy link
Copy Markdown
Collaborator

Problem description

Due to the way the BtBn ffmpeg build CI is written, there is a ~2-3 hours gap in the day, starting around 1 PM CEST, where the latest release URL points to the previous build rather than the latest, a drift that causes the download to fail in our CI, as the files in the "latest" release do not have the correct name.

Proposed solution

To alleviate this, the PR installs a pinned version of ffmpeg from BtBn which should be valid for roughly 2 years, limiting the impact for build downloads. (Includes a note to update in two years.)

Alternatives

We could revert to Chocolatey or Winget, but with some retry logic to account for sporadic download failures.

C-Achard added 2 commits April 8, 2026 16:42
Pin the Windows FFmpeg installer to a specific monthly BtbN build (FFMPEG_TAG) and switch from the "latest" releases download to the tagged /download/$tag URL. Harden PowerShell file operations by using -LiteralPath for Test-Path, Remove-Item, Get-Content, Get-FileHash, Expand-Archive and Get-ChildItem, add safer Remove-Item handling, and fail if no extracted FFmpeg directory is found. These changes improve reproducibility and make the downloader more robust against path/filename edge cases.
@C-Achard C-Achard self-assigned this Apr 8, 2026
@C-Achard C-Achard added dependencies Pull requests that update a dependency file CI Related to CI/CD jobs and automated testing labels Apr 8, 2026
@C-Achard
C-Achard marked this pull request as ready for review April 9, 2026 07:17
@C-Achard
C-Achard requested a review from deruyter92 April 9, 2026 07:17
@deruyter92

Copy link
Copy Markdown
Collaborator

@C-Achard agreed as discussed, thanks for implementing.

We should merge this one as soon as possible, as it obstructs running CI tests a few hours per day.

@C-Achard
C-Achard requested review from AlexEMG and MMathisLab April 10, 2026 09:07
@AlexEMG
AlexEMG merged commit 69116fd into main Apr 10, 2026
18 checks passed
@AlexEMG
AlexEMG deleted the cy/stable-ci-ffmpeg branch April 10, 2026 10:48
@deruyter92 deruyter92 mentioned this pull request May 21, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Related to CI/CD jobs and automated testing dependencies Pull requests that update a dependency file high-priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants