Re: [PATCH] docs: correct improper information about domain states in virsh manpage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 2017-06-06 at 12:12 -0400, Laine Stump wrote:
> Commit 24d4a0a1f removed the non-existent "dying" state from the list
> of possible domain states given in the virsh manpage, but didn't
> correct the count of states from 8 down to 7. This patch fixes that
> mismatch, while wording the sentence in a more readable/truthful
> manner.
> ---
>  tools/virsh.pod | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/virsh.pod b/tools/virsh.pod
> index aee9646..611ea88 100644
> --- a/tools/virsh.pod
> +++ b/tools/virsh.pod
> @@ -418,8 +418,9 @@ State is the run state (see below).
>  
>  B<STATES>
>  
> -The State field lists 8 states for a domain, and which ones the
> -current domain is in.
> +The State field lists what state each domain is currently in. A domain
> +can be in one of the following seven possible states:
> +
>  
>  =over 4
>  

Why don't we get rid of the number altogether while we're
at it, so that we don't risk the sentence being off sync
with the list ever again? :)

Whether or not you decide to drop the number:

Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux