[v2] iotest.FilePath fixes and cleanups

[PATCH v2 0/5] iotest.FilePath fixes and cleanups

Nir Soffer posted 5 patches 3 years, 8 months ago

Diff against v3
Download series mbox

Test docker-quick@centos7 passed

Test docker-mingw@fedora passed

Test checkpatch passed

Test FreeBSD passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200820235427.374645-1-nsoffer@redhat.com

Maintainers: Kevin Wolf <kwolf@redhat.com>, Max Reitz <mreitz@redhat.com>

There is a newer version of this series

tests/qemu-iotests/194        |  4 +--
tests/qemu-iotests/208        |  2 +-
tests/qemu-iotests/222        |  2 +-
tests/qemu-iotests/257        | 10 +++----
tests/qemu-iotests/iotests.py | 50 +++++++++++++++++------------------
5 files changed, 33 insertions(+), 35 deletions(-)

Expand all Fold all

[PATCH v2 0/5] iotest.FilePath fixes and cleanups

Posted by Nir Soffer 3 years, 8 months ago

Fix some issues introduced when iotests.FilePaths was added and merge it back
into FilePath keeping the option to create multiple file names.

Changes since v1:
- Remove unwanted submodule change [Eric]
- Add Fixes: tag

v1 was here:
https://lists.nongnu.org/archive/html/qemu-block/2020-08/msg00773.html

Nir Soffer (5):
  qemu-iotests: Fix FilePaths cleanup
  qemu-iotests: Fix FilePaths docstring
  qemu-iotests: Support varargs syntax in FilePaths
  qemu-iotests: Merge FilePaths and FilePath
  qemu-iotests: Simplify FilePath __init__

 tests/qemu-iotests/194        |  4 +--
 tests/qemu-iotests/208        |  2 +-
 tests/qemu-iotests/222        |  2 +-
 tests/qemu-iotests/257        | 10 +++----
 tests/qemu-iotests/iotests.py | 50 +++++++++++++++++------------------
 5 files changed, 33 insertions(+), 35 deletions(-)

-- 
2.26.2

Re: [PATCH v2 0/5] iotest.FilePath fixes and cleanups

Posted by Kevin Wolf 3 years, 8 months ago

Am 21.08.2020 um 01:54 hat Nir Soffer geschrieben:
> Fix some issues introduced when iotests.FilePaths was added and merge it back
> into FilePath keeping the option to create multiple file names.

Reviewed-by: Kevin Wolf <kwolf@redhat.com>