Re: [PATCH 3/4] Allow tests to be disabled

[Manolo de Medici]

The current patch misses this bit.

diff --git a/meson_options.txt b/meson_options.txt
index 0a99a059ec..9a2acd6f83 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -305,6 +305,8 @@ option('libvduse', type: 'feature', value: 'auto',
        description: 'build VDUSE Library')
 option('vduse_blk_export', type: 'feature', value: 'auto',
        description: 'VDUSE block export support')
+option('tests', type : 'feature', value : 'auto',
+       description: 'Tests build support')

 option('capstone', type: 'feature', value: 'auto',
        description: 'Whether and how to find the capstone library')

On Wed, Jan 17, 2024 at 1:32 PM Manolo de Medici
<manolodemedici@gmail.com> wrote:
>
> tests/qtest/tpm-* compilation is not disabled by disable-tpm,
> for this reason compilation fails on systems that doesn't
> support the linux/bsd TPM api. Fix this by allowing tests
> to be disabled.
>
> Signed-off-by: Manolo de Medici <manolo.demedici@gmail.com>
> ---
>  configure | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/configure b/configure
> index 21ab9a64e9..c36ce1f7e2 100755
> --- a/configure
> +++ b/configure
> @@ -254,6 +254,7 @@ skip_meson=no
>  use_containers="yes"
>  gdb_bin=$(command -v "gdb-multiarch" || command -v "gdb")
>  gdb_arches=""
> +tests="auto"
>
>  # Don't accept a target_list environment variable.
>  unset target_list
> @@ -741,6 +742,10 @@ for opt do
>    ;;
>    --disable-cfi) cfi=false
>    ;;
> +  --disable-tests) tests="disabled"
> +  ;;
> +  --enable-tests) tests="enabled"
> +  ;;
>    --disable-download) download="disabled"; git_submodules_action=validate;
>    ;;
>    --enable-download) download="enabled"; git_submodules_action=update;
> @@ -887,6 +892,7 @@ cat << EOF
>    linux-user      all linux usermode emulation targets
>    bsd-user        all BSD usermode emulation targets
>    pie             Position Independent Executables
> +  tests           build tests
>
>  NOTE: The object files are built at the place where configure is launched
>  EOF
> @@ -1792,6 +1798,7 @@ if test "$skip_meson" = no; then
>    # QEMU options
>    test "$cfi" != false && meson_option_add "-Dcfi=$cfi" "-Db_lto=$cfi"
>    test "$docs" != auto && meson_option_add "-Ddocs=$docs"
> +  test "$tests" != auto && meson_option_add "-Dtests=$tests"
>    test -n "${LIB_FUZZING_ENGINE+xxx}" && meson_option_add
> "-Dfuzzing_engine=$LIB_FUZZING_ENGINE"
>    test "$plugins" = yes && meson_option_add "-Dplugins=true"
>    test "$tcg" != enabled && meson_option_add "-Dtcg=$tcg"
> --
> 2.43.0
>
> root@hurd:~/qemu# cat 0004-Add-the-GNU-Hurd-as-a-target-host.patch
> From 5d7bf83c00168cede72a03d87d7546a0018b82a5 Mon Sep 17 00:00:00 2001
> Message-ID: 
> <5d7bf83c00168cede72a03d87d7546a0018b82a5.1705439227.git.manolo.demedici@gmail.com>
> In-Reply-To: <cover.1705439227.git.manolo.demedici@gmail.com>
> References: <cover.1705439227.git.manolo.demedici@gmail.com>
> From: Manolo de Medici <manolo.demedici@gmail.com>
> Date: Tue, 16 Jan 2024 20:12:50 +0100
> Subject: [PATCH 4/4] Add the GNU/Hurd as a target host
>
> Signed-off-by: Manolo de Medici <manolo.demedici@gmail.com>
> ---
>  configure | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/configure b/configure
> index c36ce1f7e2..d4648af96a 100755
> --- a/configure
> +++ b/configure
> @@ -354,6 +354,8 @@ elif check_define __NetBSD__; then
>    host_os=netbsd
>  elif check_define __APPLE__; then
>    host_os=darwin
> +elif check_define __GNU__; then
> +  host_os=hurd
>  else
>    # This is a fatal error, but don't report it yet, because we
>    # might be going to just print the --help text, or it might
> --
> 2.43.0