[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/4] tests/vm: Pass verbose flag into VM make
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/4] tests/vm: Pass verbose flag into VM make commands |
Date: |
Wed, 11 Jul 2018 12:15:06 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.0 |
On 07/11/2018 11:18 AM, Fam Zheng wrote:
> Our Makefile has:
>
> vm-build-%: tests/vm/%.img
> $(call quiet-command, \
> $(SRC_PATH)/tests/vm/$* \
> $(if $(V)$(DEBUG), --debug) \
> $(if $(DEBUG), --interactive) \
>
> the intention of which is to let the make command in VM have V=1 if
> V=1 is set. We pass this to the BUILD_SCRIPT format.
>
> Signed-off-by: Fam Zheng <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> tests/vm/basevm.py | 3 ++-
> tests/vm/freebsd | 4 ++--
> tests/vm/netbsd | 4 ++--
> tests/vm/openbsd | 4 ++--
> tests/vm/ubuntu.i386 | 4 ++--
> 5 files changed, 10 insertions(+), 9 deletions(-)
>
> diff --git a/tests/vm/basevm.py b/tests/vm/basevm.py
> index e5d6a328d5..25361c6b7d 100755
> --- a/tests/vm/basevm.py
> +++ b/tests/vm/basevm.py
> @@ -240,7 +240,8 @@ def main(vmcls):
> vm.add_source_dir(args.build_qemu)
> cmd = [vm.BUILD_SCRIPT.format(
> configure_opts = " ".join(argv),
> - jobs=args.jobs)]
> + jobs=args.jobs,
> + verbose="1" if args.debug else "")]
> else:
> cmd = argv
> img = args.image
> diff --git a/tests/vm/freebsd b/tests/vm/freebsd
> index 039dad8f69..b20612a6c9 100755
> --- a/tests/vm/freebsd
> +++ b/tests/vm/freebsd
> @@ -23,8 +23,8 @@ class FreeBSDVM(basevm.BaseVM):
> cd $(mktemp -d /var/tmp/qemu-test.XXXXXX);
> tar -xf /dev/vtbd1;
> ./configure {configure_opts};
> - gmake -j{jobs};
> - gmake check;
> + gmake -j{jobs} V={verbose};
> + gmake check V={verbose};
> """
>
> def build_image(self, img):
> diff --git a/tests/vm/netbsd b/tests/vm/netbsd
> index 3972d8b45c..3f9d34a208 100755
> --- a/tests/vm/netbsd
> +++ b/tests/vm/netbsd
> @@ -23,8 +23,8 @@ class NetBSDVM(basevm.BaseVM):
> cd $(mktemp -d /var/tmp/qemu-test.XXXXXX);
> tar -xf /dev/rld1a;
> ./configure --python=python2.7 {configure_opts};
> - gmake -j{jobs};
> - gmake check;
> + gmake -j{jobs} V={verbose};
> + gmake check V={verbose};
> """
>
> def build_image(self, img):
> diff --git a/tests/vm/openbsd b/tests/vm/openbsd
> index 6ae16d97fd..3285c1abde 100755
> --- a/tests/vm/openbsd
> +++ b/tests/vm/openbsd
> @@ -23,9 +23,9 @@ class OpenBSDVM(basevm.BaseVM):
> cd $(mktemp -d /var/tmp/qemu-test.XXXXXX);
> tar -xf /dev/rsd1c;
> ./configure --cc=x86_64-unknown-openbsd6.1-gcc-4.9.4
> --python=python2.7 {configure_opts};
> - gmake -j{jobs};
> + gmake -j{jobs} V={verbose};
> # XXX: "gmake check" seems to always hang or fail
> - #gmake check;
> + #gmake check V={verbose};
> """
>
> def build_image(self, img):
> diff --git a/tests/vm/ubuntu.i386 b/tests/vm/ubuntu.i386
> index fc319e0e6e..b4eaa0dedc 100755
> --- a/tests/vm/ubuntu.i386
> +++ b/tests/vm/ubuntu.i386
> @@ -25,8 +25,8 @@ class UbuntuX86VM(basevm.BaseVM):
> sudo chmod a+r /dev/vdb;
> tar -xf /dev/vdb;
> ./configure {configure_opts};
> - make -j{jobs};
> - make check;
> + make -j{jobs} V={verbose};
> + make check V={verbose};
> """
>
> def _gen_cloud_init_iso(self):
>
- [Qemu-devel] [PATCH v2 0/4] Add a CentOS test image to run docker tests, Fam Zheng, 2018/07/11
- [Qemu-devel] [PATCH v2 1/4] tests: Add an option for snapshot (default: off), Fam Zheng, 2018/07/11
- [Qemu-devel] [PATCH v2 2/4] tests/vm: Pass verbose flag into VM make commands, Fam Zheng, 2018/07/11
- Re: [Qemu-devel] [PATCH v2 2/4] tests/vm: Pass verbose flag into VM make commands,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH v2 3/4] tests: Allow overriding archive path with SRC_ARCHIVE, Fam Zheng, 2018/07/11
- [Qemu-devel] [PATCH v2 4/4] tests: Add centos VM testing, Fam Zheng, 2018/07/11
- Re: [Qemu-devel] [PATCH v2 0/4] Add a CentOS test image to run docker tests, Paolo Bonzini, 2018/07/12