Skip to content

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.

@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