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

Update references to AVChannelLayout for compatibility with FFmpeg 7 #1035

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jbucklin
Copy link

Some of the AVCodec interface has changed with FFmpeg 7. This PR updates references to AVChannelLayout properties to resolve the compile errors and replaces a call to the deprecated avcodec_close function with avcodec_free_context.

@dainisp
Copy link

dainisp commented Jun 24, 2024

Please merge, because archlinux AUR package depends on it

@zquestz
Copy link

zquestz commented Jun 26, 2024

Would be awesome to prioritize this PR, and get things building with the latest ffmpeg.

It might be nice to actually improve the PR and somehow detect if it should use the old code path, that way it works for older versions of ffmpeg as well.

Then the patch doesn't break older setups.

@Havunen
Copy link

Havunen commented Jun 26, 2024

This looks very similar to this PR: #1031? Should we try to maintain compatibility with older versions?

@Havunen
Copy link

Havunen commented Jun 26, 2024

This PR also needs this: #934

@Havunen
Copy link

Havunen commented Jun 26, 2024

I combined these 2 PR for a single patch file: #1036

Tested it builds, microphone works, audio works, recording works ( whole screen, WebM, VP8 codec )

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.

4 participants