[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/5] block: qemu-iotests - fix image cleanup
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/5] block: qemu-iotests - fix image cleanup when using spaced pathnames |
Date: |
Thu, 10 Apr 2014 08:48:10 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 |
On 04/10/2014 08:43 AM, Eric Blake wrote:
> On 04/10/2014 06:53 AM, Jeff Cody wrote:
>
>>>> +++ b/tests/qemu-iotests/common.rc
>>>> @@ -178,10 +178,10 @@ _rm_test_img()
>>>> local img=$1
>>>
>>> Since we are quoting $img, should we quote $1 as well?
>>>
>
> http://austingroupbugs.net/view.php?id=351
>
> But even with the notion of an assignment-context argument added to a
> future version of POSIX, the reality is that given the present standard,
> it's safer to either use "" to ensure no word splitting:
Well, if you were trying to be portable to multiple shells, then it
would matter. But as this script is explicitly being run under
/bin/bash, and as bash already has support for declaration utilities
where local is one such utility, your script as written is safe without
"" in the arguments to local. So I'm fine whether you choose to change
it in a respin or to leave it as written in this version.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v2 3/5] block: qemu-iotests - test for live migration, (continued)
[Qemu-devel] [PATCH v2 5/5] block: qemu-iotests: make test 019 and 086 work with spaced pathnames, Jeff Cody, 2014/04/09