qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2] tests/qemu-iotests: Rework the checks and spots using GNU


From: Philippe Mathieu-Daudé
Subject: Re: [PATCH v2] tests/qemu-iotests: Rework the checks and spots using GNU sed
Date: Wed, 16 Feb 2022 16:14:52 +0100
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.5.1

On 16/2/22 13:54, Thomas Huth wrote:
Instead of failing the iotests if GNU sed is not available (or skipping
them completely in the check-block.sh script), it would be better to
simply skip the bash-based tests that rely on GNU sed, so that the other
tests could still be run. Thus we now explicitely use "gsed" (either as
direct program or as a wrapper around "sed" if it's the GNU version)
in the spots that rely on the GNU sed behavior. Statements that use the
"-r" parameter of sed have been switched to use "-E" instead, since this
switch is supported by all sed versions on our supported build hosts
(most also support "-r", but macOS' sed only supports "-E"). With all
these changes in place, we then can also remove the sed checks from the
check-block.sh script, so that "make check-block" can now be run on
systems without GNU sed, too.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
  I've checked that this still works fine with "make vm-build-freebsd",
  "make vm-build-netbsd" and "make vm-build-openbsd" and the Cirrus-CI
  macOS tasks.

  tests/check-block.sh             | 12 ------
  tests/qemu-iotests/271           |  2 +-
  tests/qemu-iotests/common.filter | 65 ++++++++++++++++----------------
  tests/qemu-iotests/common.rc     | 45 +++++++++++-----------
  4 files changed, 57 insertions(+), 67 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>





reply via email to

[Prev in Thread] Current Thread [Next in Thread]