Skip to content

chore(cmake): set options changes as cache and with force#2762

Merged
toktok-releaser merged 1 commit intoTokTok:masterfrom
Green-Sky:cmake_option_visibility
Nov 6, 2024
Merged

chore(cmake): set options changes as cache and with force#2762
toktok-releaser merged 1 commit intoTokTok:masterfrom
Green-Sky:cmake_option_visibility

Conversation

@Green-Sky
Copy link
Copy Markdown
Member

@Green-Sky Green-Sky commented Jul 16, 2024

Without this, changes will not propergate to a parent if toxcore is used as a subdirectory.


This change is Reviewable

@Green-Sky Green-Sky requested a review from sudden6 July 16, 2024 09:29
@Green-Sky Green-Sky added this to the v0.2.20 milestone Jul 16, 2024
@Green-Sky
Copy link
Copy Markdown
Member Author

Green-Sky commented Jul 16, 2024

Alternatively one can use PARENT_SCOPE, but that looks more like a hack to me, since options are CACHE values.

OR we split toxcore properly into multiple libraries, but that's more of an v0.3 scope change.

Without this, changes will not propergate to a parent if
toxcore is used as a subdirectory.
@Green-Sky Green-Sky force-pushed the cmake_option_visibility branch from cca1738 to e9076f4 Compare November 6, 2024 12:03
@toktok-releaser toktok-releaser merged commit e9076f4 into TokTok:master Nov 6, 2024
@Green-Sky Green-Sky deleted the cmake_option_visibility branch May 16, 2025 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants