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

Fix build issue with recent ffmpeg #701

Closed

davide125 wants to merge 2 commits into mickelson:master from davide125:avcodec

Contributor

davide125 commented Feb 21, 2022 •

edited

Loading

Recent versions of ffmpeg expect a const AVCodec ** type for av_find_best_stream (e.g. see https://ffmpeg.org/doxygen/trunk/group__lavf__decoding.html#ga8d4609a8f685ad894c1503ffd1b610b4). Without this change the build fails with:

src/media.cpp: In member function ‘bool FeMedia::open(const std::string&, const std::string&, sf::Texture*)’:
src/media.cpp:1066:33: error: invalid conversion from ‘AVCodec**’ to ‘const AVCodec**’ [-fpermissive]
 1066 |                         -1, -1, &dec, 0 );
      |                                 ^~~~
      |                                 |
      |                                 AVCodec**

Tested with ffmpeg 5.0 on Linux.

davide125 and others added 2 commits

February 21, 2022 13:58


          Fix build issue with recent ffmpeg

c27312b


          Merge branch 'master' into avcodec

Owner

mickelson commented Feb 21, 2022

Hi thanks, I merged your pull request here: c27312b

I'm not sure why its not showing up as merged here

mickelson closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet