[libvirt] [PATCH] virsh: Improve change-media success message
Laine Stump
laine at laine.org
Mon Apr 6 19:29:03 UTC 2015
On 04/06/2015 02:03 PM, Cole Robinson wrote:
> $ sudo virsh change-media f19 hdc /mnt/data/devel/media/Fedora-16-x86_64-Live-KDE.iso
> succeeded to complete action update on media
>
> Change the message to:
>
> Successfully {inserted,ejected,changed} media.
>
> https://bugzilla.redhat.com/show_bug.cgi?id=967946
> ---
> tools/virsh-domain.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
ACK.
>
> diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
> index 020a308..928360c 100644
> --- a/tools/virsh-domain.c
> +++ b/tools/virsh-domain.c
> @@ -12295,6 +12295,7 @@ cmdChangeMedia(vshControl *ctl, const vshCmd *cmd)
> bool ret = false;
> vshUpdateDiskXMLType update_type;
> const char *action = NULL;
> + const char *success_msg = NULL;
> bool config = vshCommandOptBool(cmd, "config");
> bool live = vshCommandOptBool(cmd, "live");
> bool current = vshCommandOptBool(cmd, "current");
> @@ -12314,16 +12315,19 @@ cmdChangeMedia(vshControl *ctl, const vshCmd *cmd)
> if (eject) {
> update_type = VSH_UPDATE_DISK_XML_EJECT;
> action = "eject";
> + success_msg = _("Successfully ejected media.");
> }
>
> if (insert) {
> update_type = VSH_UPDATE_DISK_XML_INSERT;
> action = "insert";
> + success_msg = _("Successfully inserted media.");
> }
>
> if (update || (!eject && !insert)) {
> update_type = VSH_UPDATE_DISK_XML_UPDATE;
> action = "update";
> + success_msg = _("Successfully updated media.");
> }
>
> VSH_EXCLUSIVE_OPTIONS_VAR(current, live);
> @@ -12367,7 +12371,7 @@ cmdChangeMedia(vshControl *ctl, const vshCmd *cmd)
> goto cleanup;
> }
>
> - vshPrint(ctl, _("succeeded to complete action %s on media\n"), action);
> + vshPrint(ctl, "%s", success_msg);
> }
>
> ret = true;
More information about the libvir-list
mailing list