[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v2 2/2] block: test 'blockdev-snaps
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v2 2/2] block: test 'blockdev-snapshot' using a file BDS as the overlay |
Date: |
Tue, 03 Nov 2015 10:45:22 +0100 |
User-agent: |
Notmuch/0.13.2 (http://notmuchmail.org) Emacs/23.2.1 (i486-pc-linux-gnu) |
On Mon 02 Nov 2015 06:29:14 PM CET, Eric Blake <address@hidden> wrote:
>>> @@ -103,7 +103,8 @@ function add_snapshot_image()
>>> { 'options':
>>> { 'driver': 'qcow2', 'node-name': 'snap_"${1}"',
>>> "${extra_params}"
>>> 'file':
>>> - { 'driver': 'file', 'filename': '"${snapshot_file}"' } } }
>>> }"
>>> + { 'driver': 'file', 'filename': '"${snapshot_file}"',
>>> + 'node-name': 'file_"${1}"' } } } }"
>>
>> Pre-existing, but do those "" actually do anything?
>>
>
> Actually, the "" are wrong. Look at the full context: we have:
>
> cmd="..."${snapshot_file}"..."
>
> which means the expansion of $snapshot_file is _unquoted_.
Not really, it's quoted in all cases:
'node-name': 'snap_"${1}"'
'filename': '"${snapshot_file}"'
'node-name': 'file_"${1}"'
But it's true that the double quotes don't do anything so I'll remove
them.
Berto