* [PATCH] meson: be strict for boolean options
@ 2022-08-13 12:15 Anton Kochkov
2022-08-16 0:46 ` Philippe Mathieu-Daudé via
0 siblings, 1 reply; 3+ messages in thread
From: Anton Kochkov @ 2022-08-13 12:15 UTC (permalink / raw)
To: qemu-devel; +Cc: Anton Kochkov
While Meson buildsystem accepts the 'false' as a value
for boolean options, it's not covered by the specification
and in general invalid. Some alternative Meson implementations,
like Muon, do not accept 'false' or 'true' as a valid value
for the boolean options.
See https://mesonbuild.com/Build-options.html
Signed-off-by: Anton Kochkov <anton.kochkov@proton.me>
---
meson_options.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/meson_options.txt b/meson_options.txt
index e58e158396..63f0725174 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -82,9 +82,9 @@ option('tcg', type: 'feature', value: 'enabled',
description: 'TCG support')
option('tcg_interpreter', type: 'boolean', value: false,
description: 'TCG with bytecode interpreter (slow)')
-option('cfi', type: 'boolean', value: 'false',
+option('cfi', type: 'boolean', value: false,
description: 'Control-Flow Integrity (CFI)')
-option('cfi_debug', type: 'boolean', value: 'false',
+option('cfi_debug', type: 'boolean', value: false,
description: 'Verbose errors in case of CFI violation')
option('multiprocess', type: 'feature', value: 'auto',
description: 'Out of process device emulation support')
--
2.37.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] meson: be strict for boolean options
2022-08-13 12:15 [PATCH] meson: be strict for boolean options Anton Kochkov
@ 2022-08-16 0:46 ` Philippe Mathieu-Daudé via
0 siblings, 0 replies; 3+ messages in thread
From: Philippe Mathieu-Daudé via @ 2022-08-16 0:46 UTC (permalink / raw)
To: Anton Kochkov, qemu-devel; +Cc: Paolo Bonzini, Marc-André Lureau
On 13/8/22 14:15, Anton Kochkov wrote:
> While Meson buildsystem accepts the 'false' as a value
> for boolean options, it's not covered by the specification
> and in general invalid. Some alternative Meson implementations,
> like Muon, do not accept 'false' or 'true' as a valid value
> for the boolean options.
>
> See https://mesonbuild.com/Build-options.html
>
> Signed-off-by: Anton Kochkov <anton.kochkov@proton.me>
> ---
> meson_options.txt | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meson_options.txt b/meson_options.txt
> index e58e158396..63f0725174 100644
> --- a/meson_options.txt
> +++ b/meson_options.txt
> @@ -82,9 +82,9 @@ option('tcg', type: 'feature', value: 'enabled',
> description: 'TCG support')
> option('tcg_interpreter', type: 'boolean', value: false,
> description: 'TCG with bytecode interpreter (slow)')
> -option('cfi', type: 'boolean', value: 'false',
> +option('cfi', type: 'boolean', value: false,
> description: 'Control-Flow Integrity (CFI)')
> -option('cfi_debug', type: 'boolean', value: 'false',
> +option('cfi_debug', type: 'boolean', value: false,
> description: 'Verbose errors in case of CFI violation')
> option('multiprocess', type: 'feature', value: 'auto',
> description: 'Out of process device emulation support')
> --
> 2.37.1
>
>
>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] meson: be strict for boolean options
@ 2021-12-13 2:41 Антон Кочков
0 siblings, 0 replies; 3+ messages in thread
From: Антон Кочков @ 2021-12-13 2:41 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial
This patch allows to proceed further to be able to build with Muon buildsystem
https://sr.ht/~lattis/muon/
There are still few bugs remain, but they are on the Muon side:
https://todo.sr.ht/~lattis/muon/21
Best regards,
Anton Kochkov.
From fa80e0c17b14b8f5067d13ad7bc63e0d2cbb94ce Mon Sep 17 00:00:00 2001
From: Anton Kochkov <anton.kochkov@gmail.com>
Date: Fri, 10 Dec 2021 21:10:34 +0800
Subject: [PATCH] meson: be strict for boolean options
While Meson buildsystem accepts the 'false' as a value
for boolean options, it's not covered by the specification
and in general invalid. Some alternative Meson implementations,
like Muon, do not accept 'false' or 'true' as a valid value
for the boolean options.
See https://mesonbuild.com/Build-options.html
Signed-off-by: Anton Kochkov <anton.kochkov@gmail.com>
---
meson_options.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/meson_options.txt b/meson_options.txt
index e392323732..4ca770c1bd 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -60,9 +60,9 @@ option('tcg', type: 'feature', value: 'auto',
description: 'TCG support')
option('tcg_interpreter', type: 'boolean', value: false,
description: 'TCG with bytecode interpreter (slow)')
-option('cfi', type: 'boolean', value: 'false',
+option('cfi', type: 'boolean', value: false,
description: 'Control-Flow Integrity (CFI)')
-option('cfi_debug', type: 'boolean', value: 'false',
+option('cfi_debug', type: 'boolean', value: false,
description: 'Verbose errors in case of CFI violation')
option('multiprocess', type: 'feature', value: 'auto',
description: 'Out of process device emulation support')
--
2.33.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-08-16 0:49 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-13 12:15 [PATCH] meson: be strict for boolean options Anton Kochkov
2022-08-16 0:46 ` Philippe Mathieu-Daudé via
-- strict thread matches above, loose matches on Subject: below --
2021-12-13 2:41 Антон Кочков
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.