From: Radoslaw Smigielski <[email protected]>

The '--type' option of 'domdisplay' command selects a particular
graphical display scheme (vnc, spice, rdp, or dbus).

Partially-fixes: https://gitlab.com/libvirt/libvirt/-/work_items/9
Signed-off-by: Radoslaw Smigielski <[email protected]>
---
 tools/virsh-completer-domain.c | 13 +++++++++++++
 tools/virsh-completer-domain.h |  5 +++++
 tools/virsh-domain.c           |  1 +
 3 files changed, 19 insertions(+)

diff --git a/tools/virsh-completer-domain.c b/tools/virsh-completer-domain.c
index 8bdf68ac091d..4c0ddb01d855 100644
--- a/tools/virsh-completer-domain.c
+++ b/tools/virsh-completer-domain.c
@@ -1132,3 +1132,16 @@ virshDomainNetTypeCompleter(vshControl *ctl 
G_GNUC_UNUSED,
     return vshEnumComplete(VIR_DOMAIN_NET_TYPE_LAST,
                            virDomainNetTypeToString);
 }
+
+
+char **
+virshDomainDisplayTypeCompleter(vshControl *ctl G_GNUC_UNUSED,
+                                const vshCmd *cmd G_GNUC_UNUSED,
+                                unsigned int flags)
+{
+    static const char *types[] = {"vnc", "spice", "rdp", "dbus", NULL};
+
+    virCheckFlags(0, NULL);
+
+    return vshCommaStringListComplete(NULL, types);
+}
diff --git a/tools/virsh-completer-domain.h b/tools/virsh-completer-domain.h
index dfbc10acaaa8..3aab7bedae20 100644
--- a/tools/virsh-completer-domain.h
+++ b/tools/virsh-completer-domain.h
@@ -204,3 +204,8 @@ char **
 virshDomainNetTypeCompleter(vshControl *ctl,
                             const vshCmd *cmd,
                             unsigned int flags);
+
+char **
+virshDomainDisplayTypeCompleter(vshControl *ctl,
+                                const vshCmd *cmd,
+                                unsigned int flags);
diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c
index 79823072ad26..a546532b7dd2 100644
--- a/tools/virsh-domain.c
+++ b/tools/virsh-domain.c
@@ -12181,6 +12181,7 @@ static const vshCmdOptDef opts_domdisplay[] = {
     {.name = "type",
      .type = VSH_OT_STRING,
      .positional = true,
+     .completer = virshDomainDisplayTypeCompleter,
      .help = N_("select particular graphical display "
                 "(e.g. \"vnc\", \"spice\", \"rdp\", \"dbus\")")
     },
-- 
2.54.0

Reply via email to