From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:45929)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mreitz@redhat.com>) id 1ZtLpM-0005ph-Qn
	for qemu-devel@nongnu.org; Mon, 02 Nov 2015 15:35:44 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <mreitz@redhat.com>) id 1ZtLpL-0003tq-Rv
	for qemu-devel@nongnu.org; Mon, 02 Nov 2015 15:35:40 -0500
References: <cover.1446232490.git.jcody@redhat.com>
	<5637C813.2050902@redhat.com>
From: Max Reitz <mreitz@redhat.com>
Message-ID: <5637C90F.3070408@redhat.com>
Date: Mon, 2 Nov 2015 21:35:27 +0100
MIME-Version: 1.0
In-Reply-To: <5637C813.2050902@redhat.com>
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature";
	boundary="uaCC1PVUjTpvTSisjEKfLDlNH14TKbphV"
Subject: Re: [Qemu-devel] [PATCH v3 0/2] qemu-iotests: fix cleanup of
	background processes
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Jeff Cody <jcody@redhat.com>, qemu-devel@nongnu.org
Cc: kwolf@redhat.com, jsnow@redhat.com, qemu-block@nongnu.org

This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--uaCC1PVUjTpvTSisjEKfLDlNH14TKbphV
Content-Type: text/plain; charset=iso-8859-15
Content-Transfer-Encoding: quoted-printable

On 02.11.2015 21:31, Max Reitz wrote:
> On 30.10.2015 20:25, Jeff Cody wrote:
>> Changes from v2:
>>
>> * Pulled patch 2 into this series
>>
>> * Patch 1: Moved non-empty test conditionals inside
>>            pid file existance check (thanks Max)
>>
>>            Added a fix in for patch 058, for its
>>            self-launched qemu-nbd instance (thanks Max)
>>
>> * Patch 2: Only print the valgrind logs if the exit error
>>            matches (thanks Max)=20
>>
>> Changes from v1:
>>
>> * use 'read' instead of 'cat' (thanks Eric)
>> * quote variable in variable test (thanks Eric)
>>
>> Jeff Cody (2):
>>   qemu-iotests: fix cleanup of background processes
>>   qemu-iotests: fix -valgrind option for check
>>
>>  tests/qemu-iotests/039.out       | 30 +++++++++++++++++++++++++-----
>>  tests/qemu-iotests/058           | 12 ++++++++----
>>  tests/qemu-iotests/061.out       | 12 ++++++++++--
>>  tests/qemu-iotests/137.out       |  6 +++++-
>>  tests/qemu-iotests/common        |  9 ++-------
>>  tests/qemu-iotests/common.config | 32 +++++++++++++++++++++++++++++--=
-
>>  tests/qemu-iotests/common.qemu   | 18 ++++++++++++------
>>  tests/qemu-iotests/common.rc     | 18 +++++-------------
>>  8 files changed, 96 insertions(+), 41 deletions(-)
>=20
> Thanks, applied to my block tree with the three occurrences of ! -n in
> patch 1 changed to -z:

Err, of course it's the other way around, ! -z to -n.

Max


--uaCC1PVUjTpvTSisjEKfLDlNH14TKbphV
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

iQEcBAEBCAAGBQJWN8kPAAoJEDuxQgLoOKytXxgIAKiHIqJX12D4raaHiTgNLprS
Nbd+YoKwBYB2veQ7AFPfNuVgDlIsqJedB9ZmF/oMMpEaiU+ivlD5cIq0xcDDJRWj
ZmIb6gLuIRtSdJYFRl3INYU6nwhmfke99DqIUQ5Fit85lYiTAYuZ7Du1k1sOot2F
dXvwV2TZNPcNkQ9MW9PwKGFhJCZoTNtxtUjdhlHNXJBHCDI8ZfyO6WbmBTMM+nZr
ww5SZF06q24oO+V06PxcwSnNi03J1SzqBGs8cmDcuJmZSP8L19g17ox1bvJYtEPe
ajiFu/DqORt3ixgb48bTDJumtOKqXK4ap+Wq49ojVdUE1YcOsGRqFctCSDc38FE=
=Kadt
-----END PGP SIGNATURE-----

--uaCC1PVUjTpvTSisjEKfLDlNH14TKbphV--