Skip to content

Conversation

@j0sh
Copy link
Collaborator

@j0sh j0sh commented Dec 8, 2025

For some reason the docker build broke, maybe due to a package update since the ffmpeg build script has not been updated in a while.

Fix this by building NASM, ported over from the LPMS install script. For some reason the LPMS nasm also breaks in a different way (it's probably too old) so use the latest NASM release, 3.01

For some reason the docker build broke, maybe due to a runner or package update.

Updating ffmpeg itself to 8.0.1 might fix it.
@github-actions github-actions bot added the docker Pull requests that update Docker code label Dec 8, 2025
@codecov
Copy link

codecov bot commented Dec 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (master@87607ba). Learn more about missing BASE report.
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##             master       #3825   +/-   ##
============================================
  Coverage          ?   31.67700%           
============================================
  Files             ?         159           
  Lines             ?       39022           
  Branches          ?           0           
============================================
  Hits              ?       12361           
  Misses            ?       25771           
  Partials          ?         890           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 87607ba...fe5fe15. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@j0sh j0sh marked this pull request as ready for review December 9, 2025 18:19
@j0sh j0sh requested a review from victorges December 9, 2025 18:19
@j0sh j0sh mentioned this pull request Dec 9, 2025
5 tasks
@j0sh j0sh enabled auto-merge (squash) December 9, 2025 18:45
Copy link
Member

@victorges victorges left a comment

Choose a reason for hiding this comment

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

worked on my machine!

@j0sh j0sh merged commit 67b6e4d into master Dec 9, 2025
24 of 28 checks passed
@j0sh j0sh deleted the ja/fix-ffmpeg-build branch December 9, 2025 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Pull requests that update Docker code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants