[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/10] qemu-iotests: disintegrate more parts of
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 07/10] qemu-iotests: disintegrate more parts of common.config |
Date: |
Tue, 12 Sep 2017 16:25:57 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 09/12/2017 09:44 AM, Paolo Bonzini wrote:
> Split "check" parts from tests part.
>
> For the directory setup, the actual computation of directories goes
> in "check", while the sanity checks go in the tests.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> tests/qemu-iotests/common | 24 ++++++++++++++++++++
> tests/qemu-iotests/common.config | 49
> ----------------------------------------
> tests/qemu-iotests/common.qemu | 1 +
> tests/qemu-iotests/common.rc | 25 ++++++++++++++++++++
> 4 files changed, 50 insertions(+), 49 deletions(-)
>
> diff --git a/tests/qemu-iotests/common b/tests/qemu-iotests/common
> index abacc24114..ee313af92f 100644
> --- a/tests/qemu-iotests/common
> +++ b/tests/qemu-iotests/common
> @@ -48,6 +48,14 @@ set_prog_path()
> fi
> }
>
> +if [ -z "$TEST_DIR" ]; then
> + TEST_DIR=`pwd`/scratch
8-space indent looks odd compared to the rest of the file.
> @@ -153,6 +158,26 @@ else
> fi
> ORIG_TEST_IMG="$TEST_IMG"
>
> +if [ -z "$TEST_DIR" ]; then
> + TEST_DIR=`pwd`/scratch
We probably ought to clean things up to use $PWD instead of `pwd`
everywhere as a minor optimization (why fork, when bash already has the
right information for you?) - but that can be separate.
Split looks okay to me,
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH 03/10] qemu-iotests: move "check" code out of common.rc, (continued)
- [Qemu-devel] [PATCH 06/10] qemu-iotests: do not include common.rc in "check", Paolo Bonzini, 2017/09/12
- [Qemu-devel] [PATCH 01/10] qemu-iotests: remove dead code, Paolo Bonzini, 2017/09/12
- [Qemu-devel] [PATCH 07/10] qemu-iotests: disintegrate more parts of common.config, Paolo Bonzini, 2017/09/12
- Re: [Qemu-devel] [PATCH 07/10] qemu-iotests: disintegrate more parts of common.config,
Eric Blake <=
- [Qemu-devel] [PATCH 05/10] qemu-iotests: limit non-_PROG-suffixed variables to common.rc, Paolo Bonzini, 2017/09/12
- [Qemu-devel] [PATCH 09/10] qemu-iotests: get rid of $iam, Paolo Bonzini, 2017/09/12
- [Qemu-devel] [PATCH 08/10] qemu-iotests: fix uninitialized variable, Paolo Bonzini, 2017/09/12
- [Qemu-devel] [PATCH 10/10] qemu-iotests: merge "check" and "common", Paolo Bonzini, 2017/09/12
- Re: [Qemu-devel] [PATCH v2 00/10] cleanup qemu-iotests, Thomas Huth, 2017/09/13