From a8cf3e671047da6e7bf6c3c0d9a2178afeed1b54 Mon Sep 17 00:00:00 2001 From: Chris Thrasher Date: Tue, 31 Oct 2023 10:14:32 -0600 Subject: [PATCH] Mark `CATCH_CONFIG_` options as advanced These options are rather low-level and don't need to be seen in the CMake cache unless you opt into seeing all other advanced options. This removes a lot of cache entries from the screen when using a GUI or TUI to view the cache thus making it easier for users to focus on the cache variables they're more likely to change on a frequent basis. --- CMake/CatchConfigOptions.cmake | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/CMake/CatchConfigOptions.cmake b/CMake/CatchConfigOptions.cmake index 4a9294b7..6eae220d 100644 --- a/CMake/CatchConfigOptions.cmake +++ b/CMake/CatchConfigOptions.cmake @@ -18,10 +18,12 @@ macro(AddOverridableConfigOption OptionBaseName) option(CATCH_CONFIG_${OptionBaseName} "Read docs/configuration.md for details" OFF) option(CATCH_CONFIG_NO_${OptionBaseName} "Read docs/configuration.md for details" OFF) + mark_as_advanced(CATCH_CONFIG_${OptionBaseName} CATCH_CONFIG_NO_${OptionBaseName}) endmacro() macro(AddConfigOption OptionBaseName) option(CATCH_CONFIG_${OptionBaseName} "Read docs/configuration.md for details" OFF) + mark_as_advanced(CATCH_CONFIG_${OptionBaseName}) endmacro() set(_OverridableOptions @@ -79,6 +81,8 @@ endif() set(CATCH_CONFIG_DEFAULT_REPORTER "console" CACHE STRING "Read docs/configuration.md for details. The name of the reporter should be without quotes.") set(CATCH_CONFIG_CONSOLE_WIDTH "80" CACHE STRING "Read docs/configuration.md for details. Must form a valid integer literal.") +mark_as_advanced(CATCH_CONFIG_SHARED_LIBRARY CATCH_CONFIG_DEFAULT_REPORTER CATCH_CONFIG_CONSOLE_WIDTH) + # There is no good way to both turn this into a CMake cache variable, # and keep reasonable default semantics inside the project. Thus we do # not define it and users have to provide it as an outside variable.