From a414aa00ff4fe97680036ac89e4f735cae217d96 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Michael=20P=C3=B6hn?= <michael.poehn@fsfe.org>
Date: Sun, 26 Mar 2017 01:51:28 +0100
Subject: [PATCH] use overhauled mv destroy code in build.py

---
 fdroidserver/build.py | 27 ++++++---------------------
 makebuildserver       |  1 -
 2 files changed, 6 insertions(+), 22 deletions(-)

diff --git a/fdroidserver/build.py b/fdroidserver/build.py
index de8c3377..63923584 100644
--- a/fdroidserver/build.py
+++ b/fdroidserver/build.py
@@ -37,6 +37,7 @@ from . import common
 from . import net
 from . import metadata
 from . import scanner
+from . import vmtools
 from .common import FDroidPopen, SdkToolsPopen
 from .exception import FDroidException, BuildException, VCSException
 
@@ -289,9 +290,11 @@ def vm_get_clean_builder(reset=False):
     # If we can't use the existing machine for any reason, make a
     # new one from scratch.
     if not vm_ok:
-        vm_destroy_builder(provider)
-
-        os.mkdir('builder')
+        if os.path.isdir('builder'):
+            vm = vmtools.get_build_vm('builder')
+            vm.destroy()
+        else:
+            os.mkdir('builder')
 
         p = subprocess.Popen(['vagrant', '--version'],
                              universal_newlines=True,
@@ -348,24 +351,6 @@ def vm_suspend_builder():
     subprocess.call(['vagrant', 'suspend'], cwd='builder')
 
 
-def vm_destroy_builder(provider):
-    """Savely destroy the builder vm.
-
-    """
-    logging.info("Removing broken/incomplete/unwanted build server")
-    if os.path.exists(os.path.join('builder', 'Vagrantfile')):
-        vagrant(['destroy', '-f'], cwd='builder')
-    if os.path.isdir('builder'):
-        shutil.rmtree('builder')
-    # get rid of vm and related disk images
-    FDroidPopen(('virsh', '-c', 'qemu:///system', 'destroy', 'builder_default'))
-    logging.info("...waiting a sec...")
-    time.sleep(10)
-    FDroidPopen(('virsh', '-c', 'qemu:///system', 'undefine', 'builder_default', '--nvram', '--managed-save', '--remove-all-storage', '--snapshots-metadata'))
-    logging.info("...waiting a sec...")
-    time.sleep(10)
-
-
 # Note that 'force' here also implies test mode.
 def build_server(app, build, vcs, build_dir, output_dir, log_dir, force):
     """Do a build on the builder vm.
diff --git a/makebuildserver b/makebuildserver
index f8b892e6..703cbdd6 100755
--- a/makebuildserver
+++ b/makebuildserver
@@ -14,7 +14,6 @@ import hashlib
 import yaml
 import math
 import json
-import time
 import logging
 from clint.textui import progress
 from optparse import OptionParser