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

[Francois Gouget]

Signed-off-by: Francois Gouget 
---
 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!

[Alexandre Julliard]

Francois Gouget  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...