On 10/27/2010 10:53 PM, Justin Clift wrote:
--- Note that this includes carriage returns in the output string at
Actually, \n is newline; \r is carriage return.
appropriate locations. This lets us have reasonably verbose output, with good descriptions of the purpose of each command, that also looks decent without wrapping. tools/virsh.c | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/tools/virsh.c b/tools/virsh.c index a182d4c..11efaea 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -1359,7 +1359,9 @@ cmdUndefine(vshControl *ctl, const vshCmd *cmd) */ static const vshCmdInfo info_start[] = { {"help", N_("start a (previously defined) inactive domain")}, - {"desc", N_("Start a domain.")}, + {"desc", N_("Start a domain, either from the last managedsave\n" + " state, or via a fresh boot if no managedsave state\n" + " is present.")}, {NULL, NULL} };
@@ -1462,7 +1464,10 @@ cmdSave(vshControl *ctl, const vshCmd *cmd) */ static const vshCmdInfo info_managedsave[] = { {"help", N_("managed save of a domain state")}, - {"desc", N_("Save and stop a running domain, so libvirt can restart it from the same state")}, + {"desc", N_("Save and destroy a running domain, so it can be restarted from\n" + " the same state at a later time. When the virsh 'start'\n" + " command is next run for the domain, it will automatically\n" + " be started from this saved state.")},
ACK - both descriptions look okay to me. But before you push, be sure virsh.pod also has similar wording. -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org