From 23cbf1e0d45645473d137ef31b895ee22db782bd Mon Sep 17 00:00:00 2001
From: Nan Zhang <nzhang@redhat.com>
Date: Wed, 13 Jul 2011 16:36:32 +0800
Subject: [PATCH] virsh: fix missing prompt message for 'snapshot-delete'
 command

Make the command 'virsh snapshot-delete' has the appropriate prompt
message when executing sucessful or failed.
---
 AUTHORS       | 1 +
 tools/virsh.c | 6 +++++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/AUTHORS b/AUTHORS
index 636fa63250..64c5f8d0db 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -186,6 +186,7 @@ Patches have also been contributed by:
   Alex Jia             <ajia@redhat.com>
   Oskari Saarenmaa     <os@ohmu.fi>
   Peter Krempa         <pkrempa@redhat.com>
+  Nan Zhang            <nzhang@redhat.com>
 
   [....send patches to get your name here....]
 
diff --git a/tools/virsh.c b/tools/virsh.c
index cd17f4249f..b7cea58a57 100644
--- a/tools/virsh.c
+++ b/tools/virsh.c
@@ -11534,8 +11534,12 @@ cmdSnapshotDelete(vshControl *ctl, const vshCmd *cmd)
     if (snapshot == NULL)
         goto cleanup;
 
-    if (virDomainSnapshotDelete(snapshot, flags) < 0)
+    if (virDomainSnapshotDelete(snapshot, flags) == 0) {
+        vshPrint(ctl, _("Domain snapshot %s deleted\n"), name);
+    } else {
+        vshError(ctl, _("Failed to delete snapshot %s"), name);
         goto cleanup;
+    }
 
     ret = true;
 
-- 
GitLab