Skip to content

videoio: replace deprecated ffmpeg funtions #27690

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

Closed
wants to merge 1 commit into from

Conversation

listout
Copy link

@listout listout commented Aug 20, 2025

FFMPEG 8 has removed avcodec_close and av_stream_get_side_data. We should replace them with non-deprecated functions such as avcodec_free_context and av_packet_get_side_data respectively.

Currently opencv is not building with FFMPEG 8 (pre-release). This patch attempts at fixing them.

Closes: #27688

I'm open to recommendation and feedback.

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
  • The PR is proposed to the proper branch
  • There is a reference to the original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

FFMPEG 8 has removed avcodec_close and av_stream_get_side_data.
We should replace them with non-deprecated functions such as
avcodec_free_context and av_packet_get_side_data respectively.

Currently opencv is not building with FFMPEG 8 (pre-release). This patch
attempts at fixing them.

Closes: opencv#27688
Signed-off-by: Brahmajit Das <listout@listout.xyz>
@dkurt
Copy link
Member

dkurt commented Aug 20, 2025

Thanks for the solution. Let's merge #27691 for compatibility

@dkurt dkurt closed this Aug 20, 2025
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.

2 participants