[ffmpeg] Update to 7.0. by Sibras · Pull Request #38011 · microsoft/vcpkg

Sibras · 2024-04-06T16:00:57Z

Update ffmpeg to 7.0
Fixes #37888

[*] Changes comply with the maintainer guide.
[*] SHA512s are updated for each updated download.
[*] The "supports" clause reflects platforms that may be fixed by this new version.
[*] Any fixed CI baseline entries are removed from that file.
[*] Any patches that are no longer applied are deleted from the port's directory.
[*] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[*] Only one version is added to each modified port's versions file.

Sibras · 2024-04-07T06:08:38Z

Most of the CI failures are due to avcpp or aubio failing with the latest ffmpeg. Unfortunately these projects currently dont support ffmpeg 7.0 and thus fail (which is a common theme with ffmpeg updates within vcpkg).

MonicaLiu0311 · 2024-05-28T07:42:42Z

Is there any new progress？

dg0yt · 2024-06-01T08:01:45Z

ports/ffmpeg/portfile.cmake

    set(ENV{CC} "${CC_filename}")
    string(APPEND OPTIONS " --cc=${CC_filename}")
-    #string(APPEND OPTIONS " --host_cc=${CC_filename}") ffmpeg not yet setup for cross builds?
+    string(APPEND OPTIONS " --host_cc=${CC_filename}")


There is no host_cc in vcpkg.

host_cc is a command line arg passed to ffmpeg configure script. Due to the way the portfile is calling ffmpegs configure (calld it as cross compile) it fails without this command option

It will also fail with this option except for "almost-native" cross builds.
vcpkg allows you to tell the compiler and flags for the target triplet (e.g. android, ios).
vcpkg allows you to know the host triplet and its install prefix.
vcpkg doesn't allow you to tell the compiler and flags for the host triplet.

FTR vcpkg_configure_make turns the host cc (CC_FOR_BUILD) into a no-op for VCPKG_CROSSCOMPILING.

Unfortunately someone set the cross compile flag for ffmpeg as always on in PR #23001 (see portfile line 459). I have no idea why or what they were trying to fix but I have no way of testing all the various triplets that this effects in order to work out how to disable it again.
Due to changes in ffmpegs internal configure script in 7.0 it now does additional checks on the host compiler when the cross compile flag is passed to it. This is why the line had to be uncommented as its required to pass those checks. The actual compilation though functions the same as previously.

FTR vcpkg_configure_make turns the host cc (CC_FOR_BUILD) into a no-op for VCPKG_CROSSCOMPILING.

Unfortunately the ffmpeg port doesnt use vcpkg_configure_make

It uses vcpkg_execute_required_process to run a custom bash script. The --host_cc command line option is for this bash script to pass through to ffmpegs internal configure script

ports/ffmpeg/0009-Fix-fdk-detection.patch

dg0yt · 2024-06-01T08:10:28Z

ports/ffmpeg/0024-fix-osx-host-c11.patch

+-check_host_cflags_cc -std=$stdc ctype.h "__STDC_VERSION__ >= 201112L" ||
+-    check_host_cflags_cc -std=c11 ctype.h "__STDC_VERSION__ >= 201112L" || die "Host compiler lacks C11 support"
+check_host_cflags_cc -std=$stdc ctype.h "__STDC_VERSION__ >= 201112L" ||
+    check_host_cflags_cc -std=c11 ctype.h "__STDC_VERSION__ >= 201112L"


I don't see a change in the first line.
I wonder about the second line: Is the check wrong for osx, or doesn't it get the required flags?
But we dont have a host_cc anyways, so probably the latter.

As mentioned previously the hostcc is passed via command line option (and is just set to regular cc).
Im not sure why git diff created the patch the way it did but the only change is the removal of the "die" line as it fails in the osx CI otherwise (I dont have osx to local debug why)

Sibras · 2024-06-01T17:11:04Z

Is there any new progress？

avcpp has been updated and now works with ffmepg 7 but aubio still fails as it hasnt been updated. This was also a blocker for the 6 update as it doesnt get updated (the last aubio release was over 5 years ago) so without a patch for aubio its still failing in the CI

Neumann-A · 2024-06-01T17:28:44Z

does ffmpeg have a migration guide somewhere? Maybe its easy to patch.

Neumann-A · 2024-06-01T18:00:29Z

Does this help:
https://gitlab.archlinux.org/archlinux/packaging/packages/aubio/-/blob/main/ffmpeg7.patch?ref_type=heads

Neumann-A · 2024-06-01T18:01:37Z

https://gitlab.archlinux.org/archlinux/packaging/packages/aubio/-/blob/main/.SRCINFO?ref_type=heads

also downloads some extra patches

Sibras · 2024-06-01T18:41:35Z

Does this help: https://gitlab.archlinux.org/archlinux/packaging/packages/aubio/-/blob/main/ffmpeg7.patch?ref_type=heads

So it turns out aubio port wasnt using release versions anymore anyway so i bumbped to latest commit on master and then added that patch and it seems now to be good

…g-config (#39077) - Setup and use pkg-config for ffmpeg dependencies. #38011 (comment). - Export actual c++ link libraries for fdk-aac via pkg-config. (Same pattern as lerc, geos.) - Rectify link libraries in pkg-config for alsa, libsrt, snappy, x265. - Burn-in dllimport for libsrt and x265. - Pass detected STRIP to ffmpeg. Fixes #36852.

Sibras · 2024-06-30T08:10:57Z

Is it possible to add support for D3D12VA as well as you're into this PR? I was trying to approach it on a different PR ( see #39322 ) but we figured out it's a 7.0+ feature. The diff looks like this basically:

Thank you in advance 🙏

Done

Sibras · 2024-06-30T08:11:37Z

ffmpeg 7.0.1 has been released. It would be nice adding support for 7.0.1 while you already work on this. @Sibras

7.0.1 should be an easy update. Just need to get ther CI clear with 7.0 first

dg0yt · 2024-07-07T17:03:39Z

ports/aubio/vcpkg.json

  "homepage": "https://github.com/aubio/aubio",
  "license": "GPL-3.0-or-later",
-  "supports": "!xbox",
+  "supports": "!xbox & !osx",


aubio is healthy on osx. ffmpeg and its dependencies are not. Until #39703.

Aubio was failing to link on osx (and I have no way to debug osx). It was the only one failing to link so I disabled it. If #39703 fixes it then ill revert the change

#39703 fixes it. ffmpeg and its vcpkg port hardcode assumptions about libstdc++ which break libc++ systems such as osx and android.

The actuall error for aubio with ffmepg 7 was missing a framework link option causing a link error (was missing opengl and one other that i cant remember) which wasnt related to libstc

Maybe this was an issue, too. libstdc++ wasn't the only problem with the current ffmpeg, and I wouldn't be surprised if 7.0 adds other challenges.

This reverts commit 6d49154.

MonicaLiu0311 · 2024-08-02T02:09:59Z

Except tensorflow,alsa, other features are tested successfully in the following triplet:

x64-windows

Except tesseract,tensorflow,avisynthplus,alsa, other features are tested successfully in the following triplet:

x64-windows-static

Except tensorflow,alsa,dav1d, other features are tested successfully in the following triplet:

x86-windows

versions/a-/aubio.json

BillyONeal · 2024-08-02T17:56:19Z

Thanks for the update!

…g-config (#39077) - Setup and use pkg-config for ffmpeg dependencies. microsoft/vcpkg#38011 (comment). - Export actual c++ link libraries for fdk-aac via pkg-config. (Same pattern as lerc, geos.) - Rectify link libraries in pkg-config for alsa, libsrt, snappy, x265. - Burn-in dllimport for libsrt and x265. - Pass detected STRIP to ffmpeg. Fixes microsoft/vcpkg#36852.

Sibras added 2 commits April 7, 2024 02:57

[ffmpeg] Update to 7.0.

eb63780

Update versions.

2f09d05

WangWeiLin-MV assigned MonicaLiu0311 Apr 7, 2024

MonicaLiu0311 added the category:port-update The issue is with a library, which is requesting update new revision label Apr 7, 2024

[ffmpeg] Fix x86 nasm compile.

994ee1e

Sibras added 2 commits April 7, 2024 16:12

[ffmpeg] Fix OSX cross compilation.

b806e24

[ffmpeg] Bump version.

ecd27c2

Sibras force-pushed the ffmpeg7 branch from 869bebf to ecd27c2 Compare April 7, 2024 06:12

Sibras added 2 commits April 7, 2024 17:10

fixup

71dfa2f

Update ffmpeg.json

f36fda9

ldm0 mentioned this pull request May 3, 2024

Compiliation error after upgrading to 0.15.0 larksuite/rsmpeg#180

Closed

MonicaLiu0311 marked this pull request as draft May 28, 2024 07:42

Sibras added 2 commits June 1, 2024 17:57

Merge branch 'master' into ffmpeg7

87a1340

Update ffmpeg.json

bbc4155

dg0yt reviewed Jun 1, 2024

View reviewed changes

Sibras added 2 commits June 1, 2024 18:58

[ffmpeg] fix chromium patch

d5a8237

Update ffmpeg.json

8cde7cb

dg0yt mentioned this pull request Jun 1, 2024

[alsa,fdk-aac,ffmpeg,libsrt,snappy,x265] Code cleanup, fix and use pkg-config #39077

Merged

Sibras added 2 commits June 2, 2024 04:39

[aubio] Update to latest.

8a79791

[aubio] update version.

b9da0f3

julianxhokaxhiu mentioned this pull request Jun 17, 2024

[ffmpeg] Enable D3D12VA on Windows #39322

Closed

7 tasks

Sibras added 2 commits June 30, 2024 18:09

[ffmpeg] disable opencl on osx.

7a64773

Update ffmpeg.json

7a37c8d

Sibras added 2 commits July 6, 2024 18:38

[aubio] disable on osx.

6d49154

Update aubio.json

d995406

Sibras marked this pull request as ready for review July 7, 2024 06:00

dg0yt reviewed Jul 7, 2024

View reviewed changes

Sibras added 8 commits July 28, 2024 16:48

Merge remote-tracking branch 'upstream' into ffmpeg7

6ac1283

Update ffmpeg.json

cbc1c8b

simplify osx patch.

1145360

Update ffmpeg.json

30d85e4

Revert "[aubio] disable on osx."

466fc07

This reverts commit 6d49154.

Update aubio.json

f89edc7

fix patch

6d89823

Update ffmpeg.json

c6c57cb

dg0yt mentioned this pull request Jul 28, 2024

[ffmpeg] Add feature harfbuzz #40105

Merged

7 tasks

Merge branch 'master' of https://github.com/microsoft/vcpkg into ffmpeg7

2103055

Monica added 2 commits August 1, 2024 19:14

remove port-version

4e5b531

update git-tree

c2f3437

dg0yt reviewed Aug 2, 2024

View reviewed changes

versions/a-/aubio.json Outdated Show resolved Hide resolved

format

0ea67f8

MonicaLiu0311 approved these changes Aug 2, 2024

View reviewed changes

MonicaLiu0311 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Aug 2, 2024

dg0yt mentioned this pull request Aug 2, 2024

[ffmpeg] why disable sdl2 on osx? #40232

Closed

BillyONeal merged commit c66e3c2 into microsoft:master Aug 2, 2024

Sibras deleted the ffmpeg7 branch August 3, 2024 04:12

Conversation

Sibras commented Apr 6, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Sibras commented Apr 7, 2024

Uh oh!

MonicaLiu0311 commented May 28, 2024

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Sibras Jun 1, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Sibras Jun 1, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Sibras commented Jun 1, 2024

Uh oh!

Neumann-A commented Jun 1, 2024

Uh oh!

Neumann-A commented Jun 1, 2024

Uh oh!

Neumann-A commented Jun 1, 2024

Uh oh!

Sibras commented Jun 1, 2024

Uh oh!

Sibras commented Jun 30, 2024

Uh oh!

Sibras commented Jun 30, 2024

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

MonicaLiu0311 commented Aug 2, 2024

Uh oh!

Uh oh!

BillyONeal commented Aug 2, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

Sibras commented Apr 6, 2024 •

edited

Loading

Sibras Jun 1, 2024 •

edited

Loading

Sibras Jun 1, 2024 •

edited

Loading