[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH for-2.10 3/7] qemu-iotests/141: Fix
From: |
Jeff Cody |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH for-2.10 3/7] qemu-iotests/141: Fix image cleanup |
Date: |
Fri, 28 Jul 2017 17:04:14 -0400 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Jul 28, 2017 at 03:14:48PM +0200, Kevin Wolf wrote:
> qemu-iotests 141 attempted to use brace expansion to remove all images
> with a single command. However, for this to work, the braces shouldn't
> be quoted.
>
> With this fix, the tests correctly cleans up its scratch images.
>
> Signed-off-by: Kevin Wolf <address@hidden>
> ---
> tests/qemu-iotests/141 | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/qemu-iotests/141 b/tests/qemu-iotests/141
> index 40a3405968..2f9d7b9bc2 100755
> --- a/tests/qemu-iotests/141
> +++ b/tests/qemu-iotests/141
> @@ -31,7 +31,7 @@ _cleanup()
> {
> _cleanup_qemu
> _cleanup_test_img
> - rm -f "$TEST_DIR/{b,m,o}.$IMGFMT"
> + rm -f "$TEST_DIR"/{b,m,o}.$IMGFMT
> }
> trap "_cleanup; exit \$status" 0 1 2 3 15
>
> --
> 2.13.3
>
>
Reviewed-by: Jeff Cody <address@hidden>
- [Qemu-block] [PATCH for-2.10 0/7] qemu-iotests: Fix leaked temporary file fixes, Kevin Wolf, 2017/07/28
- [Qemu-block] [PATCH for-2.10 2/7] qemu-iotests: Remove blkdebug.conf after tests, Kevin Wolf, 2017/07/28
- [Qemu-block] [PATCH for-2.10 1/7] qemu-iotests/041: Fix leaked scratch images, Kevin Wolf, 2017/07/28
- [Qemu-block] [PATCH for-2.10 3/7] qemu-iotests/141: Fix image cleanup, Kevin Wolf, 2017/07/28
- Re: [Qemu-block] [Qemu-devel] [PATCH for-2.10 3/7] qemu-iotests/141: Fix image cleanup,
Jeff Cody <=
- [Qemu-block] [PATCH for-2.10 4/7] qemu-iotests/153: Fix leaked scratch images, Kevin Wolf, 2017/07/28
- [Qemu-block] [PATCH for-2.10 5/7] qemu-iotests/162: Fix leaked temporary files, Kevin Wolf, 2017/07/28
- [Qemu-block] [PATCH for-2.10 6/7] qemu-iotests/063: Fix leaked image, Kevin Wolf, 2017/07/28
- [Qemu-block] [PATCH for-2.10 7/7] qemu-iotests/059: Fix leaked image files, Kevin Wolf, 2017/07/28
- Re: [Qemu-block] [Qemu-devel] [PATCH for-2.10 0/7] qemu-iotests: Fix leaked temporary file fixes, Eric Blake, 2017/07/28