Skip to content

Conversation

@BillyONeal
Copy link
Member

@BillyONeal BillyONeal commented Jan 30, 2026

Fixes this build error:

/mnt/vcpkg-ci/b/qtmultimedia/src/here-src-6-8440b54a87.clean/src/plugins/multimedia/ffmpeg/recordingengine/qffmpegencoderoptions.cpp:317:26: error: use of undeclared identifier 'FF_PROFILE_H264_HIGH'
  317 |         codec->profile = FF_PROFILE_H264_HIGH;
      |                          ^
/mnt/vcpkg-ci/b/qtmultimedia/src/here-src-6-8440b54a87.clean/src/plugins/multimedia/ffmpeg/recordingengine/qffmpegencoderoptions.cpp:329:26: error: use of undeclared identifier 'FF_PROFILE_HEVC_MAIN'
  329 |         codec->profile = FF_PROFILE_HEVC_MAIN;
      |                          ^
2 errors generated.

FFMPEG appears to have renamed FF_PROFILE_Xxx to AV_PROFILE_Xxx in FFmpeg/FFmpeg@8238bc0

See also #48539

Submitted upstream: https://qt-project.atlassian.net/browse/QTBUG-143747

Fixes this build error:

```console
/mnt/vcpkg-ci/b/qtmultimedia/src/here-src-6-8440b54a87.clean/src/plugins/multimedia/ffmpeg/recordingengine/qffmpegencoderoptions.cpp:317:26: error: use of undeclared identifier 'FF_PROFILE_H264_HIGH'
  317 |         codec->profile = FF_PROFILE_H264_HIGH;
      |                          ^
/mnt/vcpkg-ci/b/qtmultimedia/src/here-src-6-8440b54a87.clean/src/plugins/multimedia/ffmpeg/recordingengine/qffmpegencoderoptions.cpp:329:26: error: use of undeclared identifier 'FF_PROFILE_HEVC_MAIN'
  329 |         codec->profile = FF_PROFILE_HEVC_MAIN;
      |                          ^
2 errors generated.
```

FFMPEG appears to have renamed `FF_PROFILE_Xxx` to `AV_PROFILE_Xxx` in FFmpeg/FFmpeg@8238bc0

See also microsoft#48539
@BillyONeal
Copy link
Member Author

I cancelled the build because the qt maintainers have jumped on this quickly: https://codereview.qt-project.org/c/qt/qtmultimedia/+/708519

We should take their patch instead of mine; I'm going to wait a bit and see if that lands Soon

@BillyONeal BillyONeal marked this pull request as draft January 30, 2026 02:03
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.

1 participant