-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
[openh264] Update version to 2.3.1 #32590
base: master
Are you sure you want to change the base?
Conversation
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Looks like ffmpeg needs to be checked. 🤔 |
get_filename_component(PKGCONFIG_PATH "${PKGCONFIG}" DIRECTORY) | ||
vcpkg_add_to_path("${PKGCONFIG_PATH}") | ||
|
||
set(ENV{PKGCONFIG_PATH} "${CURRENT_INSTALLED_DIR}/lib/pkgconfig") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vcpkg_configure_meson
correctly sets PKG_CONFIG_PATH
(!) for each build type.
set(ENV{PKG_CONFIG} "${PKGCONFIG}") | ||
get_filename_component(PKGCONFIG_PATH "${PKGCONFIG}" DIRECTORY) | ||
vcpkg_add_to_path("${PKGCONFIG_PATH}") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Either setting ENV{PKG_CONFIG}
or adding to the path should be enough.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But this is also handled implicitly by vcpkg_configure_meson
via z_vcpkg_setup_pkgconfig_path
.
Fix #32586
Note: update
openh264
version to2.3.1
.No feature needs to test.
The "supports" clause reflects platforms that may be fixed by this new versionAny fixed CI baseline entries are removed from that file../vcpkg x-add-version --all
and committing the result.