[PATCH 1/3] Improve blockpull man entry

[Sebastian Mitterle]

> We should really encourage users to properly quote their command line to
avoid unintentional globbing:

Not sure I understand, is this a request to use '"name[i]"' with
single and double quotes in the manpage?

Please, note that the line you quote is not part of the patch but
sample invocations to demonstrate behavior of blockpull command
regarding mandatory positional arguments to justify v2 of this patch.

On Tue, Apr 28, 2020 at 4:33 PM Eric Blake <eblake at redhat.com> wrote:
>
> On 4/28/20 9:19 AM, Sebastian Mitterle wrote:
> >> So bandwidth indeed is a positional argument. Since all of the manpage
> >> uses the syntax we've had here originally fixing just this place would
> >> be wrong. The only fix that should be done here is to group the --bytes
> >> flag under bandwidth as specifying --bytes is mandatory.
> >
> > I think there's misunderstanding:
> >
> > # virsh blockpull fedora vda vda[1]
>
> We should really encourage users to properly quote their command line to
> avoid unintentional globbing:
>
> # virsh blockpull fedora vda "vda[1]"
>
> (Otherwise, if I 'touch vda1', the unquoted command will see the
> argument 'vda1')
>
> > error: Scaled numeric value 'vda[1]' for <--bandwidth> option is
> > malformed or out of range
> >
> > # virsh blockpull fedora vda 1024 vda[1]
> > Block Pull started
> >
> > I'll change to [bandwidth [--bytes] [base]]
> >
>
>
> --
> Eric Blake, Principal Software Engineer
> Red Hat, Inc.           +1-919-301-3226
> Virtualization:  qemu.org | libvirt.org
>


-- 
Best,
Sebastian