[libvirt] [PATCH] doc: Correct docs for iface commands

Eric Blake eblake at redhat.com
Mon Jul 18 19:24:35 UTC 2011


On 07/18/2011 05:45 AM, Osier Yang wrote:
> duplicate documents for "iface-name", lacks of document for
> "iface-mac", inconsistent option names with virsh help strings.
> ---
>   tools/virsh.pod |   16 ++++++++++------
>   1 files changed, 10 insertions(+), 6 deletions(-)
>
> diff --git a/tools/virsh.pod b/tools/virsh.pod
> index 377fac0..4f218d3 100644
> --- a/tools/virsh.pod
> +++ b/tools/virsh.pod
> @@ -1081,20 +1081,24 @@ Returns the list of active host interfaces.  If I<--all>  is specified
>   this will also include defined but inactive interfaces.  If
>   I<--inactive>  is specified only the inactive ones will be listed.
>
> -=item B<iface-name>  I<iface-MAC>
> +=item B<iface-name>  I<interface>

We have other cases of man page names differing from virsh.c names.  For 
example, 'virsh help domid' documents '<domain>' along with later 
mentioning "domain name or uuid" under OPTIONS, but 'man virsh' merely 
documents 'domid domain-name-or-uuid' in its synopsis.

I'm okay with fixing the synopsis to match virsh, but we should probably 
scrub things to fix all commands to use consistent names.

At any rate, this is strict improvement, so:

ACK.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org




More information about the libvir-list mailing list