[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 5/5] meson_options.txt: Document that "configure" reads th
From: |
Eric Blake |
Subject: |
Re: [PATCH v2 5/5] meson_options.txt: Document that "configure" reads this file, too |
Date: |
Fri, 3 Sep 2021 12:25:59 -0500 |
User-agent: |
NeoMutt/20210205-739-420e15 |
On Fri, Sep 03, 2021 at 10:13:58AM +0200, Thomas Huth wrote:
> Add a comment that meson_options.txt is also parsed by the configure
> script and that the options need a certain layout for this to succeed.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> meson_options.txt | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/meson_options.txt b/meson_options.txt
> index 2c89e79e8b..86b3c03c7d 100644
> --- a/meson_options.txt
> +++ b/meson_options.txt
> @@ -1,3 +1,9 @@
Huh - we don't have any copyright header in this file. Oh well, not your fault.
> +# Note: This file is also parsed by "configure" to automatically handle the
> +# 'feature' options for its --enable-* and --disable-* parameters. To be able
> +# to use this automation, make sure that the corresponding options() have
> their
> +# "type: 'feature'" in the very first line, and their "description:" in their
> +# second line.
> +
> option('qemu_suffix', type : 'string', value: 'qemu',
> description: 'Suffix for QEMU data/modules/config directories (can be
> empty)')
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- [PATCH v2 0/5] Use meson_options.txt in the configure script, Thomas Huth, 2021/09/03
- [PATCH v2 2/5] meson_options.txt: Switch the default value for the vnc option to 'auto', Thomas Huth, 2021/09/03
- [PATCH v2 1/5] configure: Add the possibility to read options from meson_options.txt, Thomas Huth, 2021/09/03
- [PATCH v2 4/5] configure: Get help text from meson_options.txt, Thomas Huth, 2021/09/03
- [PATCH v2 3/5] configure: Remove options that can be handled via meson_options.txt instead, Thomas Huth, 2021/09/03
- [PATCH v2 5/5] meson_options.txt: Document that "configure" reads this file, too, Thomas Huth, 2021/09/03
- Re: [PATCH v2 5/5] meson_options.txt: Document that "configure" reads this file, too,
Eric Blake <=