From: Peter Krempa <pkrempa@redhat.com>
Date: Thu, 14 Mar 2024 17:05:07 +0000 (+0100)
Subject: virsh-backup: Fix argument annotations of 'backup-begin' command
X-Git-Tag: v10.3.0-rc1~120
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d19b12a681f1d2efb22055b07f50f1caebf5cb56;p=thirdparty%2Flibvirt.git

virsh-backup: Fix argument annotations of 'backup-begin' command

Mark the 'backupxml' as positional optional and the 'checkpointxml' as
'unwanted_positional' to preserve the positional parsing quirk.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
---

diff --git a/tools/virsh-backup.c b/tools/virsh-backup.c
index 7a7834d7ff..52325b5ca0 100644
--- a/tools/virsh-backup.c
+++ b/tools/virsh-backup.c
@@ -34,11 +34,13 @@ static const vshCmdOptDef opts_backup_begin[] = {
     VIRSH_COMMON_OPT_DOMAIN_FULL(VIR_CONNECT_LIST_DOMAINS_ACTIVE),
     {.name = "backupxml",
      .type = VSH_OT_STRING,
+     .positional = true,
      .completer = virshCompletePathLocalExisting,
      .help = N_("domain backup XML"),
     },
     {.name = "checkpointxml",
      .type = VSH_OT_STRING,
+     .unwanted_positional = true,
      .completer = virshCompletePathLocalExisting,
      .help = N_("domain checkpoint XML"),
     },