Skip to content

Update ffmpeg to 7.1.1 #17017

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Update ffmpeg to 7.1.1 #17017

wants to merge 2 commits into from

Conversation

Megamouse
Copy link
Contributor

@Megamouse Megamouse commented Apr 8, 2025 •
edited
Loading

  • Updates ffmpeg submodule to 7.1.1
  • Fixes ffmpeg deprecation warnings´

closes #17461

@Megamouse Megamouse added Audio Video Build and CI Anything related to the build process and continuous integration labels Apr 8, 2025
Megamouse
Megamouse commented Apr 8, 2025
View reviewed changes
Megamouse
Megamouse commented Apr 8, 2025
View reviewed changes
@Megamouse Megamouse force-pushed the updaates branch 3 times, most recently from c5c105b to ab3c318 Compare April 8, 2025 12:14
@Megamouse Megamouse marked this pull request as draft April 9, 2025 18:55
capriots
capriots reviewed Aug 29, 2025
View reviewed changes
rpcs3/Emu/Cell/Modules/cellAtracXdec.cpp
Comment on lines 114 to 115
ensure(!(codec->capabilities & AV_CODEC_CAP_SUBFRAMES));

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
ensure(!(codec->capabilities & AV_CODEC_CAP_SUBFRAMES));

This flag is also deprecated, the check can be removed entirely.

@capriots
Copy link
Contributor

capriots commented Aug 29, 2025 •
edited
Loading

rpcs3/rpcs3/Emu/Cell/Modules/cellAtracXdec.h

Line 196 in 4800aa9

AVCodecContext* ctx;

You need to add a default value here because it isn't initialized in AtracXdecDecoder::alloc_avcodec() anymore.

@qurious-pixel qurious-pixel mentioned this pull request Sep 24, 2025
Closed
@qurious-pixel
Copy link
Contributor

@Megamouse, I was able to build this with FFMPEG 6.1 and 8.0 using commit 1540da7 and 84a4de1, by reverting the changes to media_utils.cpp

@qurious-pixel qurious-pixel mentioned this pull request Oct 13, 2025
Open
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@capriots capriots capriots left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Audio Build and CI Anything related to the build process and continuous integration Video

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fails to build with FFmpeg 8.0 (-DUSE_SYSTEM_FFMPEG=ON)

3 participants

@Megamouse @capriots @qurious-pixel