[Qemu-devel] [PATCH v5 0/2] iotests: Add test for colon handling

Max Reitz posted 2 patches 6 years, 9 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170702150510.23276-1-mreitz@redhat.com
Test FreeBSD passed
Test checkpatch passed
Test docker passed
Test s390x passed
tests/qemu-iotests/126           | 105 +++++++++++++++++++++++++++++++++++++++
tests/qemu-iotests/126.out       |  23 +++++++++
tests/qemu-iotests/common.config |  11 ++++
tests/qemu-iotests/group         |   1 +
4 files changed, 140 insertions(+)
create mode 100755 tests/qemu-iotests/126
create mode 100644 tests/qemu-iotests/126.out
[Qemu-devel] [PATCH v5 0/2] iotests: Add test for colon handling
Posted by Max Reitz 6 years, 9 months ago
v3 of v3 of "iotests: Add test for colon handling"; thus, basically, v5.

This adds an iotest for the original series "block: Fix backing paths
for filenames with colons" and fixes common.config so it works if you
have specified the qemu binaries through relative paths. As a bonus, it
makes symlinked binaries work for out-of-tree builds.


v5: Don't try to resolve QEMU_VXHS_PROG if it is empty (which it may be)
    [Eric]


git-backport-diff against v4:

Key:
[----] : patches are identical
[####] : number of functional differences between upstream/downstream patch
[down] : patch is downstream-only
The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively

001/2:[0007] [FC] 'iotests: Use absolute paths for executables'
002/2:[----] [--] 'iotests: Add test for colon handling'


Max Reitz (2):
  iotests: Use absolute paths for executables
  iotests: Add test for colon handling

 tests/qemu-iotests/126           | 105 +++++++++++++++++++++++++++++++++++++++
 tests/qemu-iotests/126.out       |  23 +++++++++
 tests/qemu-iotests/common.config |  11 ++++
 tests/qemu-iotests/group         |   1 +
 4 files changed, 140 insertions(+)
 create mode 100755 tests/qemu-iotests/126
 create mode 100644 tests/qemu-iotests/126.out

-- 
2.9.4


Re: [Qemu-devel] [PATCH v5 0/2] iotests: Add test for colon handling
Posted by Max Reitz 6 years, 9 months ago
On 2017-07-02 17:05, Max Reitz wrote:
> v3 of v3 of "iotests: Add test for colon handling"; thus, basically, v5.
> 
> This adds an iotest for the original series "block: Fix backing paths
> for filenames with colons" and fixes common.config so it works if you
> have specified the qemu binaries through relative paths. As a bonus, it
> makes symlinked binaries work for out-of-tree builds.
> 
> 
> v5: Don't try to resolve QEMU_VXHS_PROG if it is empty (which it may be)
>     [Eric]
> 
> 
> git-backport-diff against v4:
> 
> Key:
> [----] : patches are identical
> [####] : number of functional differences between upstream/downstream patch
> [down] : patch is downstream-only
> The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively
> 
> 001/2:[0007] [FC] 'iotests: Use absolute paths for executables'
> 002/2:[----] [--] 'iotests: Add test for colon handling'
> 
> 
> Max Reitz (2):
>   iotests: Use absolute paths for executables
>   iotests: Add test for colon handling
> 
>  tests/qemu-iotests/126           | 105 +++++++++++++++++++++++++++++++++++++++
>  tests/qemu-iotests/126.out       |  23 +++++++++
>  tests/qemu-iotests/common.config |  11 ++++
>  tests/qemu-iotests/group         |   1 +
>  4 files changed, 140 insertions(+)
>  create mode 100755 tests/qemu-iotests/126
>  create mode 100644 tests/qemu-iotests/126.out

Applied to my block tree:

https://github.com/XanClic/qemu/commits/block

(Let's hope it stays there this time O:-))

Max