Skip to content
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

Fix empty CMAKE_CXX_FLAGS_RELEASE if libaom local #2367

Conversation

wantehchang
Copy link
Collaborator

Fix empty CMAKE_CXX_FLAGS_RELEASE if libaom is built locally using FetchContent and AVIF_LIBYUV is OFF.

libaom sets CMAKE_CXX_FLAGS_RELEASE and other flags as CACHE variables. To undo the changes libaom makes, unset the variables as cache variables so that they can be reset to their original values as normal variables.

Fix #2365.

cmake/Modules/LocalAom.cmake Show resolved Hide resolved
@wantehchang wantehchang force-pushed the fdintino-unset-aom-cache-variables branch from 6c9f61c to 7524d87 Compare August 6, 2024 00:53
Fix empty CMAKE_CXX_FLAGS_RELEASE if libaom is built locally using
FetchContent and AVIF_LIBYUV is OFF.

libaom sets CMAKE_CXX_FLAGS_RELEASE and other flags as CACHE variables.
To undo the changes libaom makes, unset the variables as cache variables
so that they can be reset to their original values as normal variables.

Fix AOMediaCodec#2365.
@wantehchang wantehchang force-pushed the fdintino-unset-aom-cache-variables branch from 7524d87 to cb8cd94 Compare August 6, 2024 00:55
@wantehchang wantehchang merged commit 47f154a into AOMediaCodec:main Aug 6, 2024
32 checks passed
@wantehchang wantehchang deleted the fdintino-unset-aom-cache-variables branch August 6, 2024 01:42
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.

CMAKE_CXX_FLAGS_RELEASE is empty if -DAVIF_CODEC_AOM=LOCAL -DAVIF_LIBYUV=OFF is specified
3 participants