Simplify sync by cloning in a temporary directory

See elekto-io#72 and elekto-io#73

Author: mscherer

console

@@ -81,10 +81,8 @@ if __name__ == "__main__":
 
         print('# ----------- Syncing the meta with the database ----------- #')
 
-        if not os.path.exists(backend.META) or not os.path.isdir(backend.META):
-            backend.clone()
+        backend.clone()
 
-        backend.pull()
 
         print(sync(SESSION, meta.Election.all()))
         exit()

elekto/controllers/webhook.py

@@ -27,8 +27,5 @@
 @csrf.exempt
 def webhook_sync():
     backend = meta.Meta(APP.config['META'])
-    if not os.path.exists(backend.META) or not os.path.isdir(backend.META):
-        backend.clone()
-    else:
-        backend.pull()
+    backend.clone()
     return sync(SESSION, meta.Election.all())

elekto/models/meta.py

@@ -16,6 +16,8 @@
 
 import os
 import random
+import subprocess
+import tempfile
 import flask as F
 
 from datetime import datetime
@@ -32,7 +34,9 @@ class Meta:
     """
 
     def __init__(self, config):
-        self.META = os.path.abspath(config['PATH'])
+        # keep this, as it ties the directory to the object lifecycle
+        self.TMPDIR = tempfile.TemporaryDirectory("elekto")
+        self.META = self.TMPDIR.name
         self.ELECDIR = config['ELECDIR']
         self.REMOTE = config['REMOTE']
         self.BRANCH = config['BRANCH']
@@ -42,10 +46,8 @@ def __init__(self, config):
             ".format(self.META, self.META)
 
     def clone(self):
-        os.system('{} clone -b {} -- {} {}'.format(self.git, self.BRANCH, self.REMOTE, self.META))
+        subprocess.check_call('{} clone -b {} -- {} {}'.format(self.git, self.BRANCH, self.REMOTE, self.META))
 
-    def pull(self):
-        os.system('{} pull --ff-only origin {}'.format(self.pref, self.BRANCH))
 
 
 class Election(Meta):