-
Notifications
You must be signed in to change notification settings - Fork 473
[FIX] Regression bug failing to compile with ENABLE_FFMPEG #1728
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
base: master
Are you sure you want to change the base?
Conversation
Fix unresolved function reference when compiling with ENABLE_FFMPEG
Fix regression bug for compiling with ENABLE_FFMPEG and ffmpeg 5, introduced in CCExtractor#1418
Fix regression compilation ffmpeg_intgr.c to support ffmpeg 5
[FIX] Regression bug failing to compile with ENABLE_FFMPEG
There are two more errors which popped up related to this during linking that need a patch. Do you want me to add the changes to this pull or should I create a new pull for it? |
@rboy1 separate PR if its unrelated. |
Update for changes to FFMPEG 5 API
It's related to the FFMPEG 5 API changes (also builds with ffmpeg 7.1.1), updated the pull request to include those changes. I'm now able to build this without any errors. If you're good with it please go ahead and merge the changes. |
CCExtractor CI platform finished running the test files on linux. Below is a summary of the test results, when compared to test for commit 39e051b...:
Your PR breaks these cases:
NOTE: The following tests have been failing on the master branch as well as the PR:
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Check the result page for more info. |
CCExtractor CI platform finished running the test files on windows. Below is a summary of the test results, when compared to test for commit 39e051b...:
Your PR breaks these cases:
NOTE: The following tests have been failing on the master branch as well as the PR:
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Check the result page for more info. |
In raising this pull request, I confirm the following (please check boxes):
My familiarity with the project is as follows (check one):
Regression bug, ccextractor failed to build with ENABLE_FFMPEG using ffmpeg 5 libraries
Bug was introduced while patching in ref #1418 and #1479