From 66dc554d7f72860ccb4cf6fcfd0eb89df9a7564f Mon Sep 17 00:00:00 2001 From: Thomas Heijligen Date: Sat, 13 Aug 2022 12:10:05 +0200 Subject: meson: Overhaul the print_wiki option - Rename `print_wiki` to `classic_cli_print_wiki` - Make it a meson feature type - `classic_cli` must be enabled to enable `classic_cli_print_wiki` - `classic_cli_print_wiki` is disabled by default Change-Id: Ic6c959b8b64ec2756b4535bd1b3320860f836aa5 Signed-off-by: Thomas Heijligen Reviewed-on: https://review.coreboot.org/c/flashrom/+/66703 Reviewed-by: Anastasia Klimchuk Reviewed-by: Felix Singer Tested-by: build bot (Jenkins) --- meson.build | 14 ++++++++++---- meson_options.txt | 2 +- 2 files changed, 11 insertions(+), 5 deletions(-) diff --git a/meson.build b/meson.build index 634e6270..6b00ec2f 100644 --- a/meson.build +++ b/meson.build @@ -75,7 +75,7 @@ config_stlinkv3_spi = get_option('config_stlinkv3_spi') config_parade_lspcon = get_option('config_parade_lspcon') config_mediatek_i2c_spi = get_option('config_mediatek_i2c_spi') config_realtek_mst_i2c_spi = get_option('config_realtek_mst_i2c_spi') -config_print_wiki= get_option('print_wiki') +config_print_wiki= get_option('classic_cli_print_wiki') config_default_programmer_name = get_option('default_programmer_name') config_default_programmer_args = get_option('default_programmer_args') @@ -409,9 +409,15 @@ if need_serial endif endif -if config_print_wiki - srcs += files('print_wiki.c') - cargs += '-DCONFIG_PRINT_WIKI=1' + + +if config_print_wiki.enabled() + if get_option('classic_cli').disabled() + error('`classic_cli_print_wiki` can not be enabled without `classic_cli`') + else + srcs += files('print_wiki.c') + cargs += '-DCONFIG_PRINT_WIKI=1' + endif endif if config_default_programmer_name != '' diff --git a/meson_options.txt b/meson_options.txt index 850505d4..4ec4aaa8 100644 --- a/meson_options.txt +++ b/meson_options.txt @@ -1,7 +1,7 @@ option('pciutils', type : 'boolean', value : true, description : 'use pciutils') option('usb', type : 'boolean', value : true, description : 'use libusb1') option('classic_cli', type : 'feature', value : 'enabled', description : 'classic flashrom cli binary') -option('print_wiki', type : 'boolean', value : true, description : 'Print Wiki') +option('classic_cli_print_wiki', type : 'feature', value : 'disabled', description : 'Print Wiki') option('default_programmer_name', type : 'string', description : 'default programmer') option('default_programmer_args', type : 'string', description : 'default programmer arguments') option('ich_descriptors_tool', type : 'feature', value : 'auto', description : 'Build ich_descriptors_tool') -- cgit v1.2.3