diff --git a/.travis.yml b/.travis.yml index bf7c479a..8fe55874 100644 --- a/.travis.yml +++ b/.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 install android-sdk dash gnu-sed jpeg python; - sudo pip install pep8 pyflakes pylint; - sudo pip install -e .; + brew update > /dev/null; + brew install android-sdk dash gnu-sed jpeg python3; + sudo pip3 install pep8 pyflakes pylint; + 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 - python-git python-imaging python-libcloud python-logilab-astng - python-paramiko python-pip python-pyasn1 python-pyasn1-modules - python-requests python-virtualenv python-yaml rsync - pylint pep8 dash bash ruby - python-dev libjpeg-dev zlib1g-dev; + sudo apt-get -q install -y --no-install-recommends python3 python3-dev + python3-git python3-pil python3-libcloud python3-logilab-astng + python3-paramiko python3-pip python3-pyasn1 python3-pyasn1-modules + python3-requests python3-virtualenv python3-yaml rsync + pyflakes pylint3 pep8 dash bash ruby libjpeg-dev zlib1g-dev; + sudo pip3 install pylint; fi script: diff --git a/fdroid b/fdroid index 045f2aac..53cc00c5 100755 --- a/fdroid +++ b/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.]+)["'].*''', diff --git a/fdroidserver/build.py b/fdroidserver/build.py index 10f40dc4..3837694b 100644 --- a/fdroidserver/build.py +++ b/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: diff --git a/hooks/pre-commit b/hooks/pre-commit index f0e4d657..0f083d00 100755 --- a/hooks/pre-commit +++ b/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 diff --git a/jenkins-build-makebuildserver b/jenkins-build-makebuildserver index 193718b2..36df83af 100755 --- a/jenkins-build-makebuildserver +++ b/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 diff --git a/makebuildserver b/makebuildserver index dc9d1f44..48e0f197 100755 --- a/makebuildserver +++ b/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: