Skip to content
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.

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

Fix build with FFMPEG 5 #2614

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix build with FFMPEG 5 #2614

wants to merge 2 commits into from

Conversation

Vascom
Copy link

@Vascom Vascom commented Apr 4, 2022

This change allow to build with FFMPEG 5.0.

This change allow to build with FFMPEG 5.0.
@ChrisLane
Copy link

Have you not found that you need to also remove this line to allow compilation with current builds of ffmpeg?

@Vascom
Copy link
Author

Vascom commented Apr 6, 2022

What line?
I just made this change and compile megasync at rpmfusion with FFMPEG 5.

@ChrisLane
Copy link

The line was linked in my comment.

videoStream->skip_to_keyframe = true;

It causes this compile error: #2576

@Vascom
Copy link
Author

Vascom commented Apr 6, 2022

May be it need for latest release. I used 3.9.6.b to build megasync 4.6.5.0.

@Vascom
Copy link
Author

Vascom commented Apr 11, 2022

Agree. Need to remove this line.
It was done in other place of my spec-file to build megasync.

Fix build with recent FFMPEG.
@sl1pkn07
Copy link

sl1pkn07 commented May 6, 2023

now, with ffmpeg 6, AV_CODEC_CAP_TRUNCATED is deprecated and removed

FFmpeg/FFmpeg@dd846bc

@sl1pkn07
Copy link

sl1pkn07 commented May 7, 2023

patch for ffmpeg6

#2635 (comment)

@drmikecooke
Copy link

Came across this using Bookworm debian which has ffmpeg 5.

commenting out referenced line and converting the the line 316

[code]
AVCodec* decoder = avcodec_find_decoder(codecId);
[/code]

to

[code]
const AVCodec* decoder = avcodec_find_decoder(codecId);
[/code]

also seemed to be required.

for it to get past this and move on to other work (not finished yet)

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.

None yet

4 participants