[PATCH] testbot: Fix the "no snapshot" case in BackupVM.

[Francois Gouget]

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
---
 testbot/scripts/BackupVM | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/testbot/scripts/BackupVM b/testbot/scripts/BackupVM
index 2cf3320d..071d4afa 100755
--- a/testbot/scripts/BackupVM
+++ b/testbot/scripts/BackupVM
@@ -213,6 +213,8 @@ if [ -n "$opt_snapshot" -a "$snapshots" = "/$snapdir/$opt_vm/*.xml" ]
 then
     error "the '$opt_vm' VM does not seem to have snapshots! Use --no-snapshot if this is expected."
     exit 2
+else
+    snapshots=""
 fi
 
 # Backup the symbolic links but also their targets!
-- 
2.20.1

[Alexandre Julliard]

Francois Gouget <fgouget(a)codeweavers.com> writes:
> @@ -213,6 +213,8 @@ if [ -n "$opt_snapshot" -a "$snapshots" = "/$snapdir/$opt_vm/*.xml" ]
>  then
>      error "the '$opt_vm' VM does not seem to have snapshots! Use --no-snapshot if this is expected."
>      exit 2
> +else
> +    snapshots=""
>  fi

That seems to make the snapshots variable less than useful...

-- 
Alexandre Julliard
julliard(a)winehq.org