Merge branch 'master' into 'master'

post-py3 fixes

This is a collection of fixes related to running on py3 on various platforms.

See merge request !110

.travis.yml

@@ -22,21 +22,21 @@ licenses:
 # the pip thing is a hack that can go away with trusty
 install:
   - if [[ "$TRAVIS_OS_NAME" == "osx" ]]; then
-      brew update;
+      brew update > /dev/null;
-      brew install android-sdk dash gnu-sed jpeg python;
+      brew install android-sdk dash gnu-sed jpeg python3;
-      sudo pip install pep8 pyflakes pylint;
+      sudo pip3 install pep8 pyflakes pylint;
-      sudo pip install -e  .;
+      sudo pip3 install -e  .;
       sudo rm -rf fdroidserver.egg-info;
-      echo y | android --verbose update sdk --no-ui --filter platform-tools,build-tools-23.0.2;
+      echo y | android --verbose update sdk --no-ui --all --filter platform-tools,build-tools-23.0.2;
       elif [[ "$TRAVIS_OS_NAME" == "linux" ]]; then
       sudo add-apt-repository ppa:guardianproject/fdroidserver -y;
       sudo apt-get -q update -y;
-      sudo apt-get -q install -y --no-install-recommends python
+      sudo apt-get -q install -y --no-install-recommends python3 python3-dev
-      python-git python-imaging python-libcloud python-logilab-astng
+      python3-git python3-pil python3-libcloud python3-logilab-astng
-      python-paramiko python-pip python-pyasn1 python-pyasn1-modules
+      python3-paramiko python3-pip python3-pyasn1 python3-pyasn1-modules
-      python-requests python-virtualenv python-yaml rsync
+      python3-requests python3-virtualenv python3-yaml rsync
-      pylint pep8 dash bash ruby
+      pyflakes pylint3 pep8 dash bash ruby libjpeg-dev zlib1g-dev;
-      python-dev libjpeg-dev zlib1g-dev;
+      sudo pip3 install pylint;
       fi
 
 script:

fdroid

@@ -76,9 +76,11 @@ def main():
                     import subprocess
                     try:
                         output = subprocess.check_output(['git', 'describe'],
-                                                         stderr=subprocess.STDOUT)
+                                                         stderr=subprocess.STDOUT,
+                                                         universal_newlines=True)
                     except subprocess.CalledProcessError:
-                        output = 'git commit ' + subprocess.check_output(['git', 'rev-parse', 'HEAD'])
+                        output = 'git commit ' + subprocess.check_output(['git', 'rev-parse', 'HEAD'],
+                                                                         universal_newlines=True)
                 elif os.path.exists('setup.py'):
                     import re
                     m = re.search(r'''.*[\s,\(]+version\s*=\s*["']([0-9a-z.]+)["'].*''',

fdroidserver/build.py

@@ -176,6 +176,7 @@ def get_clean_vm(reset=False):
         os.mkdir('builder')
 
         p = subprocess.Popen(['vagrant', '--version'],
+                             universal_newlines=True,
                              stdout=subprocess.PIPE)
         vver = p.communicate()[0].strip().split(' ')[1]
         if vver.split('.')[0] != '1' or int(vver.split('.')[1]) < 4:

hooks/pre-commit

@@ -75,7 +75,7 @@ find_command() {
 	for suff in "3" "-python3" ""; do
 		cmd=${1}${suff}
 		if cmd_exists $cmd; then
-			echo -n $cmd
+			echo $cmd
 			return 0
 		fi
 	done

jenkins-build-makebuildserver

@@ -51,6 +51,7 @@ echo "apt_package_cache = True" >> $WORKSPACE/makebuildserver.config.py
 # this can be handled in the jenkins job, or here:
 if [ -e fdroiddata ]; then
     cd fdroiddata
+    git checkout master
     git pull
     cd ..
 else

makebuildserver

@@ -19,7 +19,8 @@ def vagrant(params, cwd=None, printout=False):
                is the stdout (and stderr) from vagrant
     """
     p = subprocess.Popen(['vagrant'] + params, cwd=cwd,
-                         stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
+                         stdout=subprocess.PIPE, stderr=subprocess.STDOUT,
+                         universal_newlines=True)
     out = ''
     if printout:
         while True: