virsh: respect -q/--quiet more

Turn various vshPrint() informative messages into vshPrintExtra(), so
they are not printed when requesting the quiet mode; neither XML/info
outputs nor the results of commands are affected.
Also change the expected outputs of the virsh-undefine test, since virsh
is invoked in quiet mode there.

Some informative messages might still be converted (and thus silenced
when in quiet mode), but this is an improvements nonetheless.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1358179

diff --git a/tests/virsh-undefine b/tests/virsh-undefine
index 6494b4adf0b5934aed2e4953ba13bc0cff9498e7..974b0c71f7c5607ed9525ad289566de2335cb9e7 100755 (executable)
--- a/tests/virsh-undefine
+++ b/tests/virsh-undefine
@@ -36,7 +36,6 @@ test $? = 0 || fail=1
 sed '/^Persistent/n; /:/d' < out1 > out
 cat <<\EOF > exp || fail=1
 Persistent:     yes
-Domain test has been undefined
 Persistent:     no
 EOF
 compare exp out || fail=1
@@ -48,7 +47,6 @@ test $? = 0 || fail=1
 sed '/^Persistent/n; /:/d' < out1 > out
 cat <<\EOF > exp || fail=1
 Persistent:     yes
-Domain 1 has been undefined
 Persistent:     no
 EOF
 compare exp out || fail=1
@@ -58,8 +56,6 @@ $abs_top_builddir/tools/virsh -q -c test:///default \
     'shutdown test; undefine test; dominfo test' > out 2>&1
 test $? = 1 || fail=1
 cat <<\EOF > expout || fail=1
-Domain test is being shutdown
-Domain test has been undefined
 error: failed to get domain 'test'
 error: Domain not found
 EOF

diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
index fc77f162f0d9701911631596c15b9fcc1d1922fa..26b1457b5a17c346a038f0361edcfcd9a358ad0a 100644 (file)
--- a/tools/virsh-domain.c
+++ b/tools/virsh-domain.c
@@ -284,7 +284,7 @@ cmdAttachDevice(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, "%s", _("Device attached successfully\n"));
+    vshPrintExtra(ctl, "%s", _("Device attached successfully\n"));
     ret = true;
 
  cleanup:
@@ -764,7 +764,7 @@ cmdAttachDisk(vshControl *ctl, const vshCmd *cmd)
     if (ret != 0) {
         vshError(ctl, "%s", _("Failed to attach disk"));
     } else {
-        vshPrint(ctl, "%s", _("Disk attached successfully\n"));
+        vshPrintExtra(ctl, "%s", _("Disk attached successfully\n"));
         functionReturn = true;
     }
 
@@ -1075,7 +1075,7 @@ cmdAttachInterface(vshControl *ctl, const vshCmd *cmd)
     if (ret != 0) {
         vshError(ctl, "%s", _("Failed to attach interface"));
     } else {
-        vshPrint(ctl, "%s", _("Interface attached successfully\n"));
+        vshPrintExtra(ctl, "%s", _("Interface attached successfully\n"));
         functionReturn = true;
     }
 
@@ -1131,9 +1131,9 @@ cmdAutostart(vshControl *ctl, const vshCmd *cmd)
     }
 
     if (autostart)
-        vshPrint(ctl, _("Domain %s marked as autostarted\n"), name);
+        vshPrintExtra(ctl, _("Domain %s marked as autostarted\n"), name);
     else
-        vshPrint(ctl, _("Domain %s unmarked as autostarted\n"), name);
+        vshPrintExtra(ctl, _("Domain %s unmarked as autostarted\n"), name);
 
     virDomainFree(dom);
     return true;
@@ -2915,7 +2915,7 @@ cmdBlockResize(vshControl *ctl, const vshCmd *cmd)
     if (virDomainBlockResize(dom, path, size, flags) < 0) {
         vshError(ctl, _("Failed to resize block device '%s'"), path);
     } else {
-        vshPrint(ctl, _("Block device '%s' is resized"), path);
+        vshPrintExtra(ctl, _("Block device '%s' is resized"), path);
         ret = true;
     }
 
@@ -3185,7 +3185,7 @@ cmdDomIfSetLink(vshControl *ctl, const vshCmd *cmd)
         vshError(ctl, _("Failed to update interface link state"));
         goto cleanup;
     } else {
-        vshPrint(ctl, "%s", _("Device updated successfully\n"));
+        vshPrintExtra(ctl, "%s", _("Device updated successfully\n"));
         ret = true;
     }
 
@@ -3424,7 +3424,7 @@ cmdSuspend(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virDomainSuspend(dom) == 0) {
-        vshPrint(ctl, _("Domain %s suspended\n"), name);
+        vshPrintExtra(ctl, _("Domain %s suspended\n"), name);
     } else {
         vshError(ctl, _("Failed to suspend domain %s"), name);
         ret = false;
@@ -3503,7 +3503,7 @@ cmdDomPMSuspend(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, _("Domain %s successfully suspended"),
+    vshPrintExtra(ctl, _("Domain %s successfully suspended"),
              virDomainGetName(dom));
 
     ret = true;
@@ -3550,8 +3550,8 @@ cmdDomPMWakeup(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, _("Domain %s successfully woken up"),
-             virDomainGetName(dom));
+    vshPrintExtra(ctl, _("Domain %s successfully woken up"),
+                  virDomainGetName(dom));
 
     ret = true;
 
@@ -3913,7 +3913,7 @@ cmdUndefine(vshControl *ctl, const vshCmd *cmd)
 
  out:
     if (rc == 0) {
-        vshPrint(ctl, _("Domain %s has been undefined\n"), name);
+        vshPrintExtra(ctl, _("Domain %s has been undefined\n"), name);
         ret = true;
     } else {
         vshError(ctl, _("Failed to undefine domain %s"), name);
@@ -3924,15 +3924,15 @@ cmdUndefine(vshControl *ctl, const vshCmd *cmd)
     if (nvols) {
         for (i = 0; i < nvols; i++) {
             if (wipe_storage) {
-                vshPrint(ctl, _("Wiping volume '%s'(%s) ... "),
-                         vols[i].target, vols[i].source);
+                vshPrintExtra(ctl, _("Wiping volume '%s'(%s) ... "),
+                              vols[i].target, vols[i].source);
                 fflush(stdout);
                 if (virStorageVolWipe(vols[i].vol, 0) < 0) {
                     vshError(ctl, _("Failed! Volume not removed."));
                     ret = false;
                     continue;
                 } else {
-                    vshPrint(ctl, _("Done.\n"));
+                    vshPrintExtra(ctl, _("Done.\n"));
                 }
             }
 
@@ -3942,8 +3942,8 @@ cmdUndefine(vshControl *ctl, const vshCmd *cmd)
                          vols[i].target, vols[i].source);
                 ret = false;
             } else {
-                vshPrint(ctl, _("Volume '%s'(%s) removed.\n"),
-                         vols[i].target, vols[i].source);
+                vshPrintExtra(ctl, _("Volume '%s'(%s) removed.\n"),
+                              vols[i].target, vols[i].source);
             }
         }
     }
@@ -4138,8 +4138,8 @@ cmdStart(vshControl *ctl, const vshCmd *cmd)
     }
 
  started:
-    vshPrint(ctl, _("Domain %s started\n"),
-             virDomainGetName(dom));
+    vshPrintExtra(ctl, _("Domain %s started\n"),
+                  virDomainGetName(dom));
 #ifndef WIN32
     if (console && !cmdRunConsole(ctl, dom, NULL, 0))
         goto cleanup;
@@ -4404,7 +4404,7 @@ cmdSave(vshControl *ctl, const vshCmd *cmd)
     virThreadJoin(&workerThread);
 
     if (ret)
-        vshPrint(ctl, _("\nDomain %s saved to %s\n"), name, to);
+        vshPrintExtra(ctl, _("\nDomain %s saved to %s\n"), name, to);
 
  cleanup:
     virDomainFree(dom);
@@ -4520,7 +4520,7 @@ cmdSaveImageDefine(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, _("State file %s updated.\n"), file);
+    vshPrintExtra(ctl, _("State file %s updated.\n"), file);
     ret = true;
 
  cleanup:
@@ -4711,7 +4711,7 @@ cmdManagedSave(vshControl *ctl, const vshCmd *cmd)
     virThreadJoin(&workerThread);
 
     if (ret)
-        vshPrint(ctl, _("\nDomain %s state saved by libvirt\n"), name);
+        vshPrintExtra(ctl, _("\nDomain %s state saved by libvirt\n"), name);
 
  cleanup:
     virDomainFree(dom);
@@ -4762,11 +4762,11 @@ cmdManagedSaveRemove(vshControl *ctl, const vshCmd *cmd)
             goto cleanup;
         }
         else
-            vshPrint(ctl, _("Removed managedsave image for domain %s"), name);
+            vshPrintExtra(ctl, _("Removed managedsave image for domain %s"), name);
     }
     else
-        vshPrint(ctl, _("Domain %s has no manage save image; removal skipped"),
-                 name);
+        vshPrintExtra(ctl, _("Domain %s has no manage save image; removal skipped"),
+                      name);
 
     ret = true;
 
@@ -5078,7 +5078,7 @@ cmdRestore(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, _("Domain restored from %s\n"), from);
+    vshPrintExtra(ctl, _("Domain restored from %s\n"), from);
     ret = true;
 
  cleanup:
@@ -5251,7 +5251,7 @@ cmdDump(vshControl *ctl, const vshCmd *cmd)
     virThreadJoin(&workerThread);
 
     if (ret)
-        vshPrint(ctl, _("\nDomain %s dumped to %s\n"), name, to);
+        vshPrintExtra(ctl, _("\nDomain %s dumped to %s\n"), name, to);
 
  cleanup:
     virDomainFree(dom);
@@ -5385,7 +5385,7 @@ cmdScreenshot(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, _("Screenshot saved to %s, with type of %s"), file, mime);
+    vshPrintExtra(ctl, _("Screenshot saved to %s, with type of %s"), file, mime);
     ret = true;
 
  cleanup:
@@ -5458,7 +5458,7 @@ cmdSetUserPassword(vshControl *ctl, const vshCmd *cmd)
     if (virDomainSetUserPassword(dom, user, password, flags) < 0)
         goto cleanup;
 
-    vshPrint(ctl, _("Password set successfully for %s in %s"), user, name);
+    vshPrintExtra(ctl, _("Password set successfully for %s in %s"), user, name);
     ret = true;
 
  cleanup:
@@ -5494,7 +5494,7 @@ cmdResume(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virDomainResume(dom) == 0) {
-        vshPrint(ctl, _("Domain %s resumed\n"), name);
+        vshPrintExtra(ctl, _("Domain %s resumed\n"), name);
     } else {
         vshError(ctl, _("Failed to resume domain %s"), name);
         ret = false;
@@ -5575,7 +5575,7 @@ cmdShutdown(vshControl *ctl, const vshCmd *cmd)
     else
         rv = virDomainShutdown(dom);
     if (rv == 0) {
-        vshPrint(ctl, _("Domain %s is being shutdown\n"), name);
+        vshPrintExtra(ctl, _("Domain %s is being shutdown\n"), name);
     } else {
         vshError(ctl, _("Failed to shutdown domain %s"), name);
         goto cleanup;
@@ -5655,7 +5655,7 @@ cmdReboot(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
 
     if (virDomainReboot(dom, flags) == 0) {
-        vshPrint(ctl, _("Domain %s is being rebooted\n"), name);
+        vshPrintExtra(ctl, _("Domain %s is being rebooted\n"), name);
     } else {
         vshError(ctl, _("Failed to reboot domain %s"), name);
         goto cleanup;
@@ -5698,7 +5698,7 @@ cmdReset(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virDomainReset(dom, 0) == 0) {
-        vshPrint(ctl, _("Domain %s was reset\n"), name);
+        vshPrintExtra(ctl, _("Domain %s was reset\n"), name);
     } else {
         vshError(ctl, _("Failed to reset domain %s"), name);
         ret = false;
@@ -7622,8 +7622,8 @@ cmdCreate(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, _("Domain %s created from %s\n"),
-             virDomainGetName(dom), from);
+    vshPrintExtra(ctl, _("Domain %s created from %s\n"),
+                  virDomainGetName(dom), from);
 #ifndef WIN32
     if (console)
         cmdRunConsole(ctl, dom, NULL, 0);
@@ -7685,8 +7685,8 @@ cmdDefine(vshControl *ctl, const vshCmd *cmd)
     VIR_FREE(buffer);
 
     if (dom != NULL) {
-        vshPrint(ctl, _("Domain %s defined from %s\n"),
-                 virDomainGetName(dom), from);
+        vshPrintExtra(ctl, _("Domain %s defined from %s\n"),
+                      virDomainGetName(dom), from);
         virDomainFree(dom);
     } else {
         vshError(ctl, _("Failed to define domain from %s"), from);
@@ -7738,7 +7738,7 @@ cmdDestroy(vshControl *ctl, const vshCmd *cmd)
        result = virDomainDestroy(dom);
 
     if (result == 0) {
-        vshPrint(ctl, _("Domain %s destroyed\n"), name);
+        vshPrintExtra(ctl, _("Domain %s destroyed\n"), name);
     } else {
         vshError(ctl, _("Failed to destroy domain %s"), name);
         ret = false;
@@ -7867,9 +7867,9 @@ cmdDesc(vshControl *ctl, const vshCmd *cmd)
 
             /* Compare original XML with edited.  Has it changed at all? */
             if (STREQ(desc, desc_edited)) {
-                vshPrint(ctl, "%s",
-                         title ? _("Domain title not changed\n") :
-                                 _("Domain description not changed\n"));
+                vshPrintExtra(ctl, "%s",
+                              title ? _("Domain title not changed\n") :
+                                      _("Domain description not changed\n"));
                 ret = true;
                 goto cleanup;
             }
@@ -7885,9 +7885,9 @@ cmdDesc(vshControl *ctl, const vshCmd *cmd)
                              _("Failed to set new domain description"));
             goto cleanup;
         }
-        vshPrint(ctl, "%s",
-                 title ? _("Domain title updated successfully") :
-                         _("Domain description updated successfully"));
+        vshPrintExtra(ctl, "%s",
+                      title ? _("Domain title updated successfully") :
+                              _("Domain description updated successfully"));
     } else {
         desc = virshGetDomainDescription(ctl, dom, title,
                                        config?VIR_DOMAIN_XML_INACTIVE:0);
@@ -7897,10 +7897,10 @@ cmdDesc(vshControl *ctl, const vshCmd *cmd)
         if (strlen(desc) > 0)
             vshPrint(ctl, "%s", desc);
         else
-            vshPrint(ctl,
-                     title ? _("No title for domain: %s") :
-                             _("No description for domain: %s"),
-                     virDomainGetName(dom));
+            vshPrintExtra(ctl,
+                          title ? _("No title for domain: %s") :
+                                  _("No description for domain: %s"),
+                          virDomainGetName(dom));
     }
 
     ret = true;
@@ -8022,15 +8022,15 @@ cmdMetadata(vshControl *ctl, const vshCmd *cmd)
             goto cleanup;
 
         if (rem)
-            vshPrint(ctl, "%s\n", _("Metadata removed"));
+            vshPrintExtra(ctl, "%s\n", _("Metadata removed"));
         else
-            vshPrint(ctl, "%s\n", _("Metadata modified"));
+            vshPrintExtra(ctl, "%s\n", _("Metadata modified"));
     } else if (edit) {
 #define EDIT_GET_XML \
         virshDomainGetEditMetadata(ctl, dom, uri, flags)
 #define EDIT_NOT_CHANGED                                        \
         do {                                                    \
-            vshPrint(ctl, "%s", _("Metadata not changed"));     \
+            vshPrintExtra(ctl, "%s", _("Metadata not changed")); \
             ret = true;                                         \
             goto edit_cleanup;                                  \
         } while (0)
@@ -8040,7 +8040,7 @@ cmdMetadata(vshControl *ctl, const vshCmd *cmd)
                               key, uri, flags) == 0)
 #include "virsh-edit.c"
 
-        vshPrint(ctl, "%s\n", _("Metadata modified"));
+        vshPrintExtra(ctl, "%s\n", _("Metadata modified"));
     } else {
         char *data;
         /* get */
@@ -9194,8 +9194,8 @@ cmdQemuAttach(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, _("Domain %s attached to pid %u\n"),
-                virDomainGetName(dom), pid_value);
+    vshPrintExtra(ctl, _("Domain %s attached to pid %u\n"),
+                  virDomainGetName(dom), pid_value);
     virDomainFree(dom);
     ret = true;
 
@@ -9723,7 +9723,7 @@ cmdDomrename(vshControl *ctl, const vshCmd *cmd)
     if (virDomainRename(dom, new_name, 0) < 0)
         goto cleanup;
 
-    vshPrint(ctl, "Domain successfully renamed\n");
+    vshPrintExtra(ctl, "Domain successfully renamed\n");
     ret = true;
 
  cleanup:
@@ -11220,7 +11220,7 @@ cmdDetachDevice(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, "%s", _("Device detached successfully\n"));
+    vshPrintExtra(ctl, "%s", _("Device detached successfully\n"));
     funcRet = true;
 
  cleanup:
@@ -11302,7 +11302,7 @@ cmdUpdateDevice(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, "%s", _("Device updated successfully\n"));
+    vshPrintExtra(ctl, "%s", _("Device updated successfully\n"));
     ret = true;
 
  cleanup:
@@ -11489,7 +11489,7 @@ cmdDetachInterface(vshControl *ctl, const vshCmd *cmd)
     if (!ret) {
         vshError(ctl, "%s", _("Failed to detach interface"));
     } else {
-        vshPrint(ctl, "%s", _("Interface detached successfully\n"));
+        vshPrintExtra(ctl, "%s", _("Interface detached successfully\n"));
     }
     VIR_FREE(doc_live);
     VIR_FREE(doc_config);
@@ -11843,7 +11843,7 @@ cmdDetachDisk(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, "%s", _("Disk detached successfully\n"));
+    vshPrintExtra(ctl, "%s", _("Disk detached successfully\n"));
     functionReturn = true;
 
  cleanup:

diff --git a/tools/virsh-interface.c b/tools/virsh-interface.c
index 99ac84ea9b6d57b85543b54b7640ad298abe0c7d..7274527a19982a6c76d1bc90cb57e71bc508a85d 100644 (file)
--- a/tools/virsh-interface.c
+++ b/tools/virsh-interface.c
@@ -540,8 +540,8 @@ cmdInterfaceDefine(vshControl *ctl, const vshCmd *cmd)
     VIR_FREE(buffer);
 
     if (iface != NULL) {
-        vshPrint(ctl, _("Interface %s defined from %s\n"),
-                 virInterfaceGetName(iface), from);
+        vshPrintExtra(ctl, _("Interface %s defined from %s\n"),
+                      virInterfaceGetName(iface), from);
         virInterfaceFree(iface);
     } else {
         vshError(ctl, _("Failed to define interface from %s"), from);
@@ -579,7 +579,7 @@ cmdInterfaceUndefine(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virInterfaceUndefine(iface) == 0) {
-        vshPrint(ctl, _("Interface %s undefined\n"), name);
+        vshPrintExtra(ctl, _("Interface %s undefined\n"), name);
     } else {
         vshError(ctl, _("Failed to undefine interface %s"), name);
         ret = false;
@@ -618,7 +618,7 @@ cmdInterfaceStart(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virInterfaceCreate(iface, 0) == 0) {
-        vshPrint(ctl, _("Interface %s started\n"), name);
+        vshPrintExtra(ctl, _("Interface %s started\n"), name);
     } else {
         vshError(ctl, _("Failed to start interface %s"), name);
         ret = false;
@@ -657,7 +657,7 @@ cmdInterfaceDestroy(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virInterfaceDestroy(iface, 0) == 0) {
-        vshPrint(ctl, _("Interface %s destroyed\n"), name);
+        vshPrintExtra(ctl, _("Interface %s destroyed\n"), name);
     } else {
         vshError(ctl, _("Failed to destroy interface %s"), name);
         ret = false;
@@ -696,7 +696,7 @@ cmdInterfaceBegin(vshControl *ctl, const vshCmd *cmd ATTRIBUTE_UNUSED)
         return false;
     }
 
-    vshPrint(ctl, "%s", _("Network config change transaction started\n"));
+    vshPrintExtra(ctl, "%s", _("Network config change transaction started\n"));
     return true;
 }
 
@@ -727,7 +727,7 @@ cmdInterfaceCommit(vshControl *ctl, const vshCmd *cmd ATTRIBUTE_UNUSED)
         return false;
     }
 
-    vshPrint(ctl, "%s", _("Network config change transaction committed\n"));
+    vshPrintExtra(ctl, "%s", _("Network config change transaction committed\n"));
     return true;
 }
 
@@ -758,7 +758,7 @@ cmdInterfaceRollback(vshControl *ctl, const vshCmd *cmd ATTRIBUTE_UNUSED)
         return false;
     }
 
-    vshPrint(ctl, "%s", _("Network config change transaction rolled back\n"));
+    vshPrintExtra(ctl, "%s", _("Network config change transaction rolled back\n"));
     return true;
 }
 

diff --git a/tools/virsh-network.c b/tools/virsh-network.c
index 0054495931f4de2bf7c58f3c42f2c8d4ba79dfb9..16980411f35fa44d39d6d1ee1852a8aad0c3bdd2 100644 (file)
--- a/tools/virsh-network.c
+++ b/tools/virsh-network.c
@@ -123,9 +123,9 @@ cmdNetworkAutostart(vshControl *ctl, const vshCmd *cmd)
     }
 
     if (autostart)
-        vshPrint(ctl, _("Network %s marked as autostarted\n"), name);
+        vshPrintExtra(ctl, _("Network %s marked as autostarted\n"), name);
     else
-        vshPrint(ctl, _("Network %s unmarked as autostarted\n"), name);
+        vshPrintExtra(ctl, _("Network %s unmarked as autostarted\n"), name);
 
     virNetworkFree(network);
     return true;
@@ -168,8 +168,8 @@ cmdNetworkCreate(vshControl *ctl, const vshCmd *cmd)
     VIR_FREE(buffer);
 
     if (network != NULL) {
-        vshPrint(ctl, _("Network %s created from %s\n"),
-                 virNetworkGetName(network), from);
+        vshPrintExtra(ctl, _("Network %s created from %s\n"),
+                      virNetworkGetName(network), from);
         virNetworkFree(network);
     } else {
         vshError(ctl, _("Failed to create network from %s"), from);
@@ -216,8 +216,8 @@ cmdNetworkDefine(vshControl *ctl, const vshCmd *cmd)
     VIR_FREE(buffer);
 
     if (network != NULL) {
-        vshPrint(ctl, _("Network %s defined from %s\n"),
-                 virNetworkGetName(network), from);
+        vshPrintExtra(ctl, _("Network %s defined from %s\n"),
+                      virNetworkGetName(network), from);
         virNetworkFree(network);
     } else {
         vshError(ctl, _("Failed to define network from %s"), from);
@@ -255,7 +255,7 @@ cmdNetworkDestroy(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virNetworkDestroy(network) == 0) {
-        vshPrint(ctl, _("Network %s destroyed\n"), name);
+        vshPrintExtra(ctl, _("Network %s destroyed\n"), name);
     } else {
         vshError(ctl, _("Failed to destroy network %s"), name);
         ret = false;
@@ -794,7 +794,7 @@ cmdNetworkStart(vshControl *ctl, const vshCmd *cmd)
          return false;
 
     if (virNetworkCreate(network) == 0) {
-        vshPrint(ctl, _("Network %s started\n"), name);
+        vshPrintExtra(ctl, _("Network %s started\n"), name);
     } else {
         vshError(ctl, _("Failed to start network %s"), name);
         ret = false;
@@ -832,7 +832,7 @@ cmdNetworkUndefine(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virNetworkUndefine(network) == 0) {
-        vshPrint(ctl, _("Network %s has been undefined\n"), name);
+        vshPrintExtra(ctl, _("Network %s has been undefined\n"), name);
     } else {
         vshError(ctl, _("Failed to undefine network %s"), name);
         ret = false;

diff --git a/tools/virsh-nodedev.c b/tools/virsh-nodedev.c
index 7676dcc486417d153bc93c32a2a6996f0ae59774..805c0ffe71d0b201b030ede322789273cc734af0 100644 (file)
--- a/tools/virsh-nodedev.c
+++ b/tools/virsh-nodedev.c
@@ -75,8 +75,8 @@ cmdNodeDeviceCreate(vshControl *ctl, const vshCmd *cmd)
     VIR_FREE(buffer);
 
     if (dev != NULL) {
-        vshPrint(ctl, _("Node device %s created from %s\n"),
-                 virNodeDeviceGetName(dev), from);
+        vshPrintExtra(ctl, _("Node device %s created from %s\n"),
+                      virNodeDeviceGetName(dev), from);
         virNodeDeviceFree(dev);
     } else {
         vshError(ctl, _("Failed to create node device from %s"), from);
@@ -148,7 +148,7 @@ cmdNodeDeviceDestroy(vshControl *ctl, const vshCmd *cmd)
     }
 
     if (virNodeDeviceDestroy(dev) == 0) {
-        vshPrint(ctl, _("Destroyed node device '%s'\n"), device_value);
+        vshPrintExtra(ctl, _("Destroyed node device '%s'\n"), device_value);
     } else {
         vshError(ctl, _("Failed to destroy node device '%s'"), device_value);
         goto cleanup;
@@ -632,7 +632,7 @@ cmdNodeDeviceDetach(vshControl *ctl, const vshCmd *cmd)
     }
 
     if (ret)
-        vshPrint(ctl, _("Device %s detached\n"), name);
+        vshPrintExtra(ctl, _("Device %s detached\n"), name);
     else
         vshError(ctl, _("Failed to detach device %s"), name);
 
@@ -680,7 +680,7 @@ cmdNodeDeviceReAttach(vshControl *ctl, const vshCmd *cmd)
     }
 
     if (virNodeDeviceReAttach(device) == 0) {
-        vshPrint(ctl, _("Device %s re-attached\n"), name);
+        vshPrintExtra(ctl, _("Device %s re-attached\n"), name);
     } else {
         vshError(ctl, _("Failed to re-attach device %s"), name);
         ret = false;
@@ -730,7 +730,7 @@ cmdNodeDeviceReset(vshControl *ctl, const vshCmd *cmd)
     }
 
     if (virNodeDeviceReset(device) == 0) {
-        vshPrint(ctl, _("Device %s reset\n"), name);
+        vshPrintExtra(ctl, _("Device %s reset\n"), name);
     } else {
         vshError(ctl, _("Failed to reset device %s"), name);
         ret = false;

diff --git a/tools/virsh-nwfilter.c b/tools/virsh-nwfilter.c
index 6e6582d4d87edac72debee6c24b208ce62fb9949..daf58a5a1a83fd0e473aa0f9617b94939669a244 100644 (file)
--- a/tools/virsh-nwfilter.c
+++ b/tools/virsh-nwfilter.c
@@ -109,8 +109,8 @@ cmdNWFilterDefine(vshControl *ctl, const vshCmd *cmd)
     VIR_FREE(buffer);
 
     if (nwfilter != NULL) {
-        vshPrint(ctl, _("Network filter %s defined from %s\n"),
-                 virNWFilterGetName(nwfilter), from);
+        vshPrintExtra(ctl, _("Network filter %s defined from %s\n"),
+                      virNWFilterGetName(nwfilter), from);
         virNWFilterFree(nwfilter);
     } else {
         vshError(ctl, _("Failed to define network filter from %s"), from);
@@ -152,7 +152,7 @@ cmdNWFilterUndefine(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virNWFilterUndefine(nwfilter) == 0) {
-        vshPrint(ctl, _("Network filter %s undefined\n"), name);
+        vshPrintExtra(ctl, _("Network filter %s undefined\n"), name);
     } else {
         vshError(ctl, _("Failed to undefine network filter %s"), name);
         ret = false;

diff --git a/tools/virsh-pool.c b/tools/virsh-pool.c
index 448927249e382be7f393ecec84d925805ac7719c..f952887b818cbebcb4b1d0328d43ae64c922413b 100644 (file)
--- a/tools/virsh-pool.c
+++ b/tools/virsh-pool.c
@@ -209,9 +209,9 @@ cmdPoolAutostart(vshControl *ctl, const vshCmd *cmd)
     }
 
     if (autostart)
-        vshPrint(ctl, _("Pool %s marked as autostarted\n"), name);
+        vshPrintExtra(ctl, _("Pool %s marked as autostarted\n"), name);
     else
-        vshPrint(ctl, _("Pool %s unmarked as autostarted\n"), name);
+        vshPrintExtra(ctl, _("Pool %s unmarked as autostarted\n"), name);
 
     virStoragePoolFree(pool);
     return true;
@@ -276,8 +276,8 @@ cmdPoolCreate(vshControl *ctl, const vshCmd *cmd)
     VIR_FREE(buffer);
 
     if (pool != NULL) {
-        vshPrint(ctl, _("Pool %s created from %s\n"),
-                 virStoragePoolGetName(pool), from);
+        vshPrintExtra(ctl, _("Pool %s created from %s\n"),
+                      virStoragePoolGetName(pool), from);
         virStoragePoolFree(pool);
     } else {
         vshError(ctl, _("Failed to create pool from %s"), from);
@@ -449,7 +449,7 @@ cmdPoolCreateAs(vshControl *ctl, const vshCmd *cmd)
         VIR_FREE(xml);
 
         if (pool != NULL) {
-            vshPrint(ctl, _("Pool %s created\n"), name);
+            vshPrintExtra(ctl, _("Pool %s created\n"), name);
             virStoragePoolFree(pool);
         } else {
             vshError(ctl, _("Failed to create pool %s"), name);
@@ -498,8 +498,8 @@ cmdPoolDefine(vshControl *ctl, const vshCmd *cmd)
     VIR_FREE(buffer);
 
     if (pool != NULL) {
-        vshPrint(ctl, _("Pool %s defined from %s\n"),
-                 virStoragePoolGetName(pool), from);
+        vshPrintExtra(ctl, _("Pool %s defined from %s\n"),
+                      virStoragePoolGetName(pool), from);
         virStoragePoolFree(pool);
     } else {
         vshError(ctl, _("Failed to define pool from %s"), from);
@@ -541,7 +541,7 @@ cmdPoolDefineAs(vshControl *ctl, const vshCmd *cmd)
         VIR_FREE(xml);
 
         if (pool != NULL) {
-            vshPrint(ctl, _("Pool %s defined\n"), name);
+            vshPrintExtra(ctl, _("Pool %s defined\n"), name);
             virStoragePoolFree(pool);
         } else {
             vshError(ctl, _("Failed to define pool %s"), name);
@@ -590,7 +590,7 @@ cmdPoolBuild(vshControl *ctl, const vshCmd *cmd)
         flags |= VIR_STORAGE_POOL_BUILD_OVERWRITE;
 
     if (virStoragePoolBuild(pool, flags) == 0) {
-        vshPrint(ctl, _("Pool %s built\n"), name);
+        vshPrintExtra(ctl, _("Pool %s built\n"), name);
     } else {
         vshError(ctl, _("Failed to build pool %s"), name);
         ret = false;
@@ -631,7 +631,7 @@ cmdPoolDestroy(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virStoragePoolDestroy(pool) == 0) {
-        vshPrint(ctl, _("Pool %s destroyed\n"), name);
+        vshPrintExtra(ctl, _("Pool %s destroyed\n"), name);
     } else {
         vshError(ctl, _("Failed to destroy pool %s"), name);
         ret = false;
@@ -671,7 +671,7 @@ cmdPoolDelete(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virStoragePoolDelete(pool, 0) == 0) {
-        vshPrint(ctl, _("Pool %s deleted\n"), name);
+        vshPrintExtra(ctl, _("Pool %s deleted\n"), name);
     } else {
         vshError(ctl, _("Failed to delete pool %s"), name);
         ret = false;
@@ -711,7 +711,7 @@ cmdPoolRefresh(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virStoragePoolRefresh(pool, 0) == 0) {
-        vshPrint(ctl, _("Pool %s refreshed\n"), name);
+        vshPrintExtra(ctl, _("Pool %s refreshed\n"), name);
     } else {
         vshError(ctl, _("Failed to refresh pool %s"), name);
         ret = false;
@@ -1732,7 +1732,7 @@ cmdPoolStart(vshControl *ctl, const vshCmd *cmd)
         flags |= VIR_STORAGE_POOL_CREATE_WITH_BUILD_NO_OVERWRITE;
 
     if (virStoragePoolCreate(pool, flags) == 0) {
-        vshPrint(ctl, _("Pool %s started\n"), name);
+        vshPrintExtra(ctl, _("Pool %s started\n"), name);
     } else {
         vshError(ctl, _("Failed to start pool %s"), name);
         ret = false;
@@ -1772,7 +1772,7 @@ cmdPoolUndefine(vshControl *ctl, const vshCmd *cmd)
         return false;
 
     if (virStoragePoolUndefine(pool) == 0) {
-        vshPrint(ctl, _("Pool %s has been undefined\n"), name);
+        vshPrintExtra(ctl, _("Pool %s has been undefined\n"), name);
     } else {
         vshError(ctl, _("Failed to undefine pool %s"), name);
         ret = false;

diff --git a/tools/virsh-secret.c b/tools/virsh-secret.c
index 537bdd7b1864b033942c67f928e2ab4a207527f6..d868eb3c5fa428b4ca1a01d52083819cad66f1ce 100644 (file)
--- a/tools/virsh-secret.c
+++ b/tools/virsh-secret.c
@@ -104,7 +104,7 @@ cmdSecretDefine(vshControl *ctl, const vshCmd *cmd)
         goto cleanup;
     }
 
-    vshPrint(ctl, _("Secret %s created\n"), uuid);
+    vshPrintExtra(ctl, _("Secret %s created\n"), uuid);
     ret = true;
 
  cleanup:
@@ -219,7 +219,7 @@ cmdSecretSetValue(vshControl *ctl, const vshCmd *cmd)
         vshError(ctl, "%s", _("Failed to set secret value"));
         goto cleanup;
     }
-    vshPrint(ctl, "%s", _("Secret value set\n"));
+    vshPrintExtra(ctl, "%s", _("Secret value set\n"));
     ret = true;
 
  cleanup:
@@ -316,7 +316,7 @@ cmdSecretUndefine(vshControl *ctl, const vshCmd *cmd)
         vshError(ctl, _("Failed to delete secret %s"), uuid);
         goto cleanup;
     }
-    vshPrint(ctl, _("Secret %s deleted\n"), uuid);
+    vshPrintExtra(ctl, _("Secret %s deleted\n"), uuid);
     ret = true;
 
  cleanup:

diff --git a/tools/virsh-snapshot.c b/tools/virsh-snapshot.c
index f879e7aec4708de948942a49cda8392ddd7b6ad2..95db0969e4084842b404da4b25224e52de595ae1 100644 (file)
--- a/tools/virsh-snapshot.c
+++ b/tools/virsh-snapshot.c
@@ -97,9 +97,9 @@ virshSnapshotCreate(vshControl *ctl, virDomainPtr dom, const char *buffer,
     }
 
     if (from)
-        vshPrint(ctl, _("Domain snapshot %s created from '%s'"), name, from);
+        vshPrintExtra(ctl, _("Domain snapshot %s created from '%s'"), name, from);
     else
-        vshPrint(ctl, _("Domain snapshot %s created"), name);
+        vshPrintExtra(ctl, _("Domain snapshot %s created"), name);
 
     ret = true;
 
@@ -575,10 +575,10 @@ cmdSnapshotEdit(vshControl *ctl, const vshCmd *cmd)
 
     edited_name = virDomainSnapshotGetName(edited);
     if (STREQ(name, edited_name)) {
-        vshPrint(ctl, _("Snapshot %s edited.\n"), name);
+        vshPrintExtra(ctl, _("Snapshot %s edited.\n"), name);
     } else if (clone_okay) {
-        vshPrint(ctl, _("Snapshot %s cloned to %s.\n"), name,
-                 edited_name);
+        vshPrintExtra(ctl, _("Snapshot %s cloned to %s.\n"), name,
+                      edited_name);
     } else {
         unsigned int delete_flags;
 
@@ -683,7 +683,7 @@ cmdSnapshotCurrent(vshControl *ctl, const vshCmd *cmd)
             goto cleanup;
 
         virDomainSnapshotFree(snapshot2);
-        vshPrint(ctl, _("Snapshot %s set as current"), snapshotname);
+        vshPrintExtra(ctl, _("Snapshot %s set as current"), snapshotname);
         ret = true;
         goto cleanup;
     }
@@ -1927,9 +1927,9 @@ cmdSnapshotDelete(vshControl *ctl, const vshCmd *cmd)
      * list of descendants.  But that's a lot of code to maintain.  */
     if (virDomainSnapshotDelete(snapshot, flags) == 0) {
         if (flags & VIR_DOMAIN_SNAPSHOT_DELETE_CHILDREN_ONLY)
-            vshPrint(ctl, _("Domain snapshot %s children deleted\n"), name);
+            vshPrintExtra(ctl, _("Domain snapshot %s children deleted\n"), name);
         else
-            vshPrint(ctl, _("Domain snapshot %s deleted\n"), name);
+            vshPrintExtra(ctl, _("Domain snapshot %s deleted\n"), name);
     } else {
         vshError(ctl, _("Failed to delete snapshot %s"), name);
         goto cleanup;

diff --git a/tools/virsh-volume.c b/tools/virsh-volume.c
index d35fee15b27ab1c4343b6f9f19cb7ff1042b1656..a2f0432462bd79ef670bc2522ef9ea3fc21e8fd9 100644 (file)
--- a/tools/virsh-volume.c
+++ b/tools/virsh-volume.c
@@ -349,7 +349,7 @@ cmdVolCreateAs(vshControl *ctl, const vshCmd *cmd)
             vshError(ctl, _("Failed to create vol %s"), name);
             goto cleanup;
         }
-        vshPrint(ctl, _("Vol %s created\n"), name);
+        vshPrintExtra(ctl, _("Vol %s created\n"), name);
     }
 
     ret = true;
@@ -411,8 +411,8 @@ cmdVolCreate(vshControl *ctl, const vshCmd *cmd)
     }
 
     if ((vol = virStorageVolCreateXML(pool, buffer, flags))) {
-        vshPrint(ctl, _("Vol %s created from %s\n"),
-                 virStorageVolGetName(vol), from);
+        vshPrintExtra(ctl, _("Vol %s created from %s\n"),
+                      virStorageVolGetName(vol), from);
         virStorageVolFree(vol);
         ret = true;
     } else {
@@ -490,8 +490,8 @@ cmdVolCreateFrom(vshControl *ctl, const vshCmd *cmd)
     newvol = virStorageVolCreateXMLFrom(pool, buffer, inputvol, flags);
 
     if (newvol != NULL) {
-        vshPrint(ctl, _("Vol %s created from input vol %s\n"),
-                 virStorageVolGetName(newvol), virStorageVolGetName(inputvol));
+        vshPrintExtra(ctl, _("Vol %s created from input vol %s\n"),
+                      virStorageVolGetName(newvol), virStorageVolGetName(inputvol));
     } else {
         vshError(ctl, _("Failed to create vol from %s"), from);
         goto cleanup;
@@ -612,8 +612,8 @@ cmdVolClone(vshControl *ctl, const vshCmd *cmd)
     newvol = virStorageVolCreateXMLFrom(origpool, (char *) newxml, origvol, flags);
 
     if (newvol != NULL) {
-        vshPrint(ctl, _("Vol %s cloned from %s\n"),
-                 virStorageVolGetName(newvol), virStorageVolGetName(origvol));
+        vshPrintExtra(ctl, _("Vol %s cloned from %s\n"),
+                      virStorageVolGetName(newvol), virStorageVolGetName(origvol));
     } else {
         vshError(ctl, _("Failed to clone vol from %s"),
                  virStorageVolGetName(origvol));
@@ -879,7 +879,7 @@ cmdVolDelete(vshControl *ctl, const vshCmd *cmd)
         flags |= VIR_STORAGE_VOL_DELETE_WITH_SNAPSHOTS;
 
     if (virStorageVolDelete(vol, flags) == 0) {
-        vshPrint(ctl, _("Vol %s deleted\n"), name);
+        vshPrintExtra(ctl, _("Vol %s deleted\n"), name);
     } else {
         vshError(ctl, _("Failed to delete vol %s"), name);
         ret = false;
@@ -950,7 +950,7 @@ cmdVolWipe(vshControl *ctl, const vshCmd *cmd)
         goto out;
     }
 
-    vshPrint(ctl, _("Vol %s wiped\n"), name);
+    vshPrintExtra(ctl, _("Vol %s wiped\n"), name);
     ret = true;
  out:
     virStorageVolFree(vol);