[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 5/7] qemu-io: allow specifying image as a set of
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 5/7] qemu-io: allow specifying image as a set of options args |
Date: |
Tue, 22 Dec 2015 10:06:00 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 12/22/2015 04:06 AM, Daniel P. Berrange wrote:
> Currently qemu-io allows an image filename to be passed on the
> command line, but does not have a way to set any options except
> the format eg
>
> qemu-io https://127.0.0.1/images/centos7.iso
> qemu-io /home/berrange/demo.qcow2
Well, you CAN set the options by specifying the file as:
qemu-io json:{...}
but that's a mouthful to type, so I don't mind adding the new syntax.
>
> This adds a --source arg (that is mutually exclusive with a
> positional filename arg and -f arg) that accepts a full option
> string, as well as the original syntax eg
>
> qemu-io --source driver=http,url=https://127.0.0.1/images,sslverify=off
> qemu-io --source https://127.0.0.1/images/centos7.iso
> qemu-io --source file=/home/berrange/demo.qcow2
> qemu-io --source /home/berrange/demo.qcow2
>
> Signed-off-by: Daniel P. Berrange <address@hidden>
> ---
> qemu-io.c | 37 ++++++++++++++++++++++++++++++++++++-
> 1 file changed, 36 insertions(+), 1 deletion(-)
>
> @@ -572,7 +590,24 @@ int main(int argc, char **argv)
> flags |= BDRV_O_RDWR;
> }
>
> - if ((argc - optind) == 1) {
> + qopts = qemu_opts_find(&file_opts, NULL);
> + if (qopts) {
> + char *file;
> + if (opts) {
> + error_report("--source and -f are mutually exclusive");
> + exit(1);
> + }
> + if ((argc - optind) == 1) {
What if I specify more than one option? Shouldn't this be >=, rather
than ==? Also, the inner () are redundant.
> + error_report("--source and filename are mutually exclusive");
> + exit(1);
> + }
> + file = g_strdup(qemu_opt_get(qopts, "file"));
> + qemu_opt_unset(qopts, "file");
> + opts = qemu_opts_to_qdict(qopts, NULL);
> + qemu_opts_reset(&file_opts);
> + openfile(file, flags, opts);
> + g_free(file);
> + } else if ((argc - optind) == 1) {
Pre-existing, but the inner () are redundant.
> openfile(argv[optind], flags, opts);
> }
> command_loop();
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 0/7] Make qemu-img/qemu-nbd/qemu-io CLI more flexible, Daniel P. Berrange, 2015/12/22
- [Qemu-devel] [PATCH 1/7] qom: add user_creatable_add & user_creatable_del methods, Daniel P. Berrange, 2015/12/22
- [Qemu-devel] [PATCH 3/7] qemu-nbd: add support for --object command line arg, Daniel P. Berrange, 2015/12/22
- [Qemu-devel] [PATCH 5/7] qemu-io: allow specifying image as a set of options args, Daniel P. Berrange, 2015/12/22
- Re: [Qemu-devel] [PATCH 5/7] qemu-io: allow specifying image as a set of options args,
Eric Blake <=
- [Qemu-devel] [PATCH 6/7] qemu-nbd: allow specifying image as a set of options args, Daniel P. Berrange, 2015/12/22
- [Qemu-devel] [PATCH 4/7] qemu-io: add support for --object command line arg, Daniel P. Berrange, 2015/12/22
[Qemu-devel] [PATCH 2/7] qemu-img: add support for --object command line arg, Daniel P. Berrange, 2015/12/22