From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40421) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aQZKD-0006Ei-BX for qemu-devel@nongnu.org; Tue, 02 Feb 2016 06:40:50 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aQZKC-0004nn-AG for qemu-devel@nongnu.org; Tue, 02 Feb 2016 06:40:49 -0500 Date: Tue, 2 Feb 2016 11:40:40 +0000 From: "Daniel P. Berrange" Message-ID: <20160202114040.GG18461@redhat.com> References: <1453815262-13440-1-git-send-email-berrange@redhat.com> <1453815262-13440-8-git-send-email-berrange@redhat.com> <20160127143029.GD9484@noname.str.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20160127143029.GD9484@noname.str.redhat.com> Subject: Re: [Qemu-devel] [PATCH v4 07/10] qemu-img: allow specifying image as a set of options args Reply-To: "Daniel P. Berrange" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Kevin Wolf Cc: qemu-block@nongnu.org, qemu-devel@nongnu.org, Markus Armbruster , Paolo Bonzini , Andreas =?utf-8?Q?F=C3=A4rber?= On Wed, Jan 27, 2016 at 03:30:29PM +0100, Kevin Wolf wrote: > Am 26.01.2016 um 14:34 hat Daniel P. Berrange geschrieben: > > Currently qemu-img allows an image filename to be passed on the > > command line, but unless using the JSON format, it does not have > > a way to set any options except the format eg > > > > qemu-img info https://127.0.0.1/images/centos7.iso > > > > This adds a --image-opts arg that indicates that the positional > > filename should be interpreted as a full option string, not > > just a filename. > > > > qemu-img info --source driver=http,url=https://127.0.0.1/images,sslverify=off > > > > This flag is mutually exclusive with the '-f' / '-F' flags. > > > > Signed-off-by: Daniel P. Berrange > > > @@ -633,7 +671,20 @@ static int img_check(int argc, char **argv) > > return 1; > > } > > > > - blk = img_open("image", filename, fmt, flags, true, quiet); > > + if (image_opts) { > > + if (fmt) { > > + error_report("--image-opts and --format are mutually exclusive"); > > + exit(1); > > + } > > + opts = qemu_opts_parse_noisily(qemu_find_opts("source"), > > + filename, true); > > + if (!opts) { > > + exit(1); > > + } > > + blk = img_open_opts("image", opts, flags); > > + } else { > > + blk = img_open_file("image", filename, fmt, flags, true, quiet); > > + } > > This block is duplicated everywhere. Can you make it a function so that > it stays a single line in each of the callers? Yes, I've done that and its much nicer now Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|