From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60997) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a9zcE-0006I2-0k for qemu-devel@nongnu.org; Fri, 18 Dec 2015 13:18:54 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a9zcD-0005vU-0D for qemu-devel@nongnu.org; Fri, 18 Dec 2015 13:18:53 -0500 References: <1450259693-25595-1-git-send-email-famz@redhat.com> From: Max Reitz Message-ID: <56744E02.10506@redhat.com> Date: Fri, 18 Dec 2015 19:18:42 +0100 MIME-Version: 1.0 In-Reply-To: <1450259693-25595-1-git-send-email-famz@redhat.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="dU67ne4LLiLvgWkpvxIHnHG23HR9GXeVe" Subject: Re: [Qemu-devel] [PATCH 00/10] iotests: Clean up "mv $TEST_IMG $TEST_IMG.XXX" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Fam Zheng , qemu-devel@nongnu.org Cc: Kevin Wolf , jcody@redhat.com, jsnow@redhat.com, qemu-block@nongnu.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --dU67ne4LLiLvgWkpvxIHnHG23HR9GXeVe Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: quoted-printable On 16.12.2015 10:54, Fam Zheng wrote: > Commit 794d00f71d fixed two "mv" commands into the TEST_IMG override ap= proach. > There are still more occasions of "mv", this series fixes them. >=20 > The benefit is it drops the assumption that the created image is a sing= le file, > which is not true for VMDK or the coming QBM format. On top of this, t= hose > formats can be added to these cases later. >=20 > The patches stems from QBM work, but I will also take a look at adding > applicable VMDK subformats once merged. >=20 > Please review. Patches 1 to 9: Reviewed-by: Max Reitz >=20 > Fam >=20 >=20 > Fam Zheng (10): > iotests: 018: Use TEST_IMG override instead of "mv" > iotests: 019: Use TEST_IMG override instead of "mv" > iotests: 020: Use TEST_IMG override instead of "mv" > iotests: 024: Use TEST_IMG override instead of "mv" > iotests: 028: Use TEST_IMG override instead of "mv" > iotests: 034: Use TEST_IMG override instead of "mv" > iotests: 037: Use TEST_IMG override instead of "mv" > iotests: 038: Use TEST_IMG override instead of "mv" > iotests: 050: Use TEST_IMG override instead of "mv" > iotests: 095: Use TEST_IMG override instead of "mv" >=20 > tests/qemu-iotests/018 | 8 ++++---- > tests/qemu-iotests/018.out | 2 +- > tests/qemu-iotests/019 | 13 +++++++------ > tests/qemu-iotests/019.out | 4 ++-- > tests/qemu-iotests/020 | 7 +++++-- > tests/qemu-iotests/020.out | 2 +- > tests/qemu-iotests/024 | 10 ++++++++-- > tests/qemu-iotests/024.out | 4 ++-- > tests/qemu-iotests/028 | 5 ++++- > tests/qemu-iotests/028.out | 2 +- > tests/qemu-iotests/034 | 6 +++++- > tests/qemu-iotests/034.out | 2 +- > tests/qemu-iotests/037 | 5 ++++- > tests/qemu-iotests/037.out | 2 +- > tests/qemu-iotests/038 | 5 ++++- > tests/qemu-iotests/038.out | 2 +- > tests/qemu-iotests/050 | 9 +++++++-- > tests/qemu-iotests/050.out | 4 ++-- > tests/qemu-iotests/095 | 12 ++++++------ > tests/qemu-iotests/095.out | 8 ++++---- > 20 files changed, 70 insertions(+), 42 deletions(-) >=20 --dU67ne4LLiLvgWkpvxIHnHG23HR9GXeVe Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJWdE4DAAoJEDuxQgLoOKytwJ8H/0CkJM2VUvX/cWIEtdi44gyf IeoXbYDPnOKFp01aXV/vSsRI+pCT69Nk3sI/MJSUzqJoUzELW50DrBThKvS9mPOe VNWcLownHYABM5nUflqGiYUpr68GrKpPK2LaKLE64HJgeKtogfmc5pR6A3kWYPkn DeJS8AF2U9GN9i6WOT0SFOo7Pngiip8a0Mdc4uDboZ+Ts7a7sf+hFcsT5Lngsg/z 6qbDrybUcR+lETkedTaAbVohw3upEcmbAYl/JN5/zeLgK9lQ3CUdqutGpec2b/gS ffXEiV+5lFhD3tE8krfm4J96mdqZNhvHGYwt3+IlZaWMcUSZO6VKPEo+9ETs4O8= =ZwlQ -----END PGP SIGNATURE----- --dU67ne4LLiLvgWkpvxIHnHG23HR9GXeVe--