Re: [PATCH] virsh: Fix flag semantics and docs for "desc" command

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

 



On 02/08/2012 04:07 PM, Eric Blake wrote:
On 02/08/2012 05:16 AM, Peter Krempa wrote:
This patch fixes the domain modification impact flags for tie virsh
desc command to match the new semantics and fix the docs to match
actual behavior.
---
  tools/virsh.c   |   21 +++++++++++++++------
  tools/virsh.pod |   12 +++++-------
  2 files changed, 20 insertions(+), 13 deletions(-)


+++ b/tools/virsh.pod
@@ -435,7 +435,7 @@ Define a domain from an XML<file>. The domain definition is registered
  but not started.  If domain is already running, the changes will take
  effect on the next boot.

-=item B<desc>  [I<--live>  | I<--config>] [I<--title>] [I<--edit>]
+=item B<desc>  [[I<--live>  I<--config>  | I<--current>] [I<--title>] [I<--edit>]

Missing some []; should be:

[[I<--live>] [I<--config>] | [I<--current>]] ...

ACK with that fixed.


Fixed & pushed. Thanks!

Peter

--
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]