From: Daniel Martí <mvdan@mvdan.cc>
Date: Mon, 4 Nov 2013 09:22:22 +0000 (+0100)
Subject: Fixes to makebuildserver and verify
X-Git-Tag: 0.1~236
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=fc804e68d2cf472e8cbe22298457c68dd4e103b0;p=fdroidserver.git

Fixes to makebuildserver and verify
---

diff --git a/fdroidserver/common.py b/fdroidserver/common.py
index 5845ecc5..9e3ca99c 100644
--- a/fdroidserver/common.py
+++ b/fdroidserver/common.py
@@ -30,17 +30,17 @@ import magic
 config = None
 options = None
 
-def read_config(opts):
+def read_config(opts, config_file='config.py'):
     """Read the repository config
 
-    The config is read from config.py, which is in the current directory when
+    The config is read from config_file, which is in the current directory when
     any of the repo management commands are used.
     """
     global config, options
 
     if config is not None:
         return config
-    if not os.path.isfile('config.py'):
+    if not os.path.isfile(config_file):
         print "Missing config file - is this a repo directory?"
         sys.exit(2)
 
@@ -58,8 +58,8 @@ def read_config(opts):
         'max_icon_size': 72,
         'stats_to_carbon': False
     }
-    print "Reading config.py..."
-    execfile('config.py', config)
+    print "Reading %s..." % config_file
+    execfile(config_file, config)
     return config
 
 
diff --git a/fdroidserver/verify.py b/fdroidserver/verify.py
index d81d4073..7b9644e2 100644
--- a/fdroidserver/verify.py
+++ b/fdroidserver/verify.py
@@ -34,8 +34,6 @@ def main():
 
     global options, config
 
-    options, args = parse_commandline()
-
     # Parse command line...
     parser = OptionParser()
     parser.add_option("-v", "--verbose", action="store_true", default=False,
diff --git a/makebuildserver.py b/makebuildserver.py
index 1075d1de..fa37bca5 100755
--- a/makebuildserver.py
+++ b/makebuildserver.py
@@ -5,8 +5,7 @@ import sys
 import subprocess
 import time
 from optparse import OptionParser
-
-execfile('makebs.config.py', globals())
+from fdroidserver import common
 
 def vagrant(params, cwd=None, printout=False):
     """Run vagrant.
@@ -43,8 +42,7 @@ parser.add_option("-c", "--clean", action="store_true", default=False,
                       help="Build from scratch, rather than attempting to update the existing server")
 options, args = parser.parse_args()
 
-
-
+config = common.read_config(options, 'makebs.config.py')
 
 if not os.path.exists('makebuildserver.py') or not os.path.exists(serverdir):
     print 'This must be run from the correct directory!'
@@ -67,7 +65,7 @@ cachefiles = [
     ('gradle-1.8-bin.zip',
      'http://services.gradle.org/distributions/gradle-1.8-bin.zip',
      'a342bbfa15fd18e2482287da4959588f45a41b60910970a16e6d97959aea5703')]
-if arch64:
+if config['arch64']:
     cachefiles.extend([
     ('android-ndk-r9-linux-x64_64.tar.bz2',
      'http://dl.google.com/android/ndk/android-ndk-r9-linux-x86_64.tar.bz2',
@@ -114,11 +112,11 @@ Vagrant::Config.run do |config|
   config.vm.customize ["modifyvm", :id, "--memory", "{2}"]
 
   config.vm.provision :shell, :path => "fixpaths.sh"
-""".format(basebox, baseboxurl, memory)
-if aptproxy:
+""".format(config['basebox'], config['baseboxurl'], config['memory'])
+if 'aptproxy' in config and config['aptproxy']:
     vagrantfile += """
   config.vm.provision :shell, :inline => 'sudo echo "Acquire::http {{ Proxy \\"{0}\\"; }};" > /etc/apt/apt.conf.d/02proxy && sudo apt-get update'
-""".format(aptproxy)
+""".format(config['aptproxy'])
 vagrantfile += """
   config.vm.provision :chef_solo do |chef|
     chef.cookbooks_path = "cookbooks"