Repository navigation
build(deps): update FFmpeg to 9.0.2 - #341
Merged
Merged
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #341 +/- ##
========================================
Coverage 65.10% 65.10%
========================================
Files 49 49
Lines 5875 5875
Branches 2621 2621
========================================
Hits 3825 3825
- Misses 969 1160 +191
+ Partials 1081 890 -191
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Harness.
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Description
Update vendored FFmpeg from 8.1.2 to
n9.0.2(946fcce07b6dcd0331c8cc609192aeff5e1924f8) and trackrelease/9.0, preserving the nxdk build.moonlight_patch_sources()function that stages fresh inputs, applies patches in an isolated build directory, tracks dependencies, and preserves unchanged output timestamps.jinc()helper intact. Declarej1(double)in the private nxdk compatibility header so the helper can be parsed without suppressing compiler diagnostics. The unused helper emits noj1reference; an actual caller would still require an implementation.Screenshot
Issues Fixed or Closed
Roadmap Issues
Testing
Validated locally on Windows using MSYS2 MinGW64:
BUILD_DOCS=ON,BUILD_TESTS=ON, andBUILD_XBOX=ON;cmake --build cmake-build-release -j4passed, including Xbox XBE/ISO packaging and the enabled video experiments.ctest --test-dir cmake-build-release --output-on-failure: 285/285 passed.cmake --build cmake-build-release --target docs: strict Dockle/Doxygenapibuild passed using the repository's existing Dockle environment, producing 438 HTML pages.uv --no-cache run --locked lb-update-clang-format: passed; unrelated formatting changes were not introduced.llvm-nm -uverified thatlibswscale.ahas noj1reference.-Werror=implicit-function-declarationremains enabled, and there is no scaler source override.git diff --check origin/master...HEAD: passed.No unit tests were added: the changes affect dependency builds and a declaration-only compatibility adapter, validated through the actual Xbox configure, compile, and link steps.
Emulator and real Xbox playback were not tested. Linux/macOS builds and hosted CI are not covered by these local results.
Type of Change
Checklist
AI Usage
See our AI usage policy.
Codex assisted with implementation, review, validation, and preparing this PR description.