Merge branch '1.0-polish' into 'master'

1.0 polish

Closes #424

See merge request fdroid/fdroidserver!405
This commit is contained in:
Hans-Christoph Steiner 2017-12-11 21:44:16 +00:00
commit 858b398927
4 changed files with 9 additions and 2 deletions

View file

@ -35,7 +35,7 @@ __by_ext() {
} }
__package() { __package() {
files="$(__by_ext txt) $(__by_ext yml) $(__by_ext json) $(__by_ext xml)" files="$(__by_ext txt) $(__by_ext yml) $(__by_ext json)"
COMPREPLY=( $( compgen -W "$files" -- $cur ) ) COMPREPLY=( $( compgen -W "$files" -- $cur ) )
} }

View file

@ -414,6 +414,11 @@ def build_local(app, build, vcs, build_dir, output_dir, log_dir, srclib_dir, ext
raise BuildException("Error running sudo command for %s:%s" % raise BuildException("Error running sudo command for %s:%s" %
(app.id, build.versionName), p.output) (app.id, build.versionName), p.output)
p = FDroidPopen(['sudo', 'apt-get', '-y', 'purge', 'sudo'])
if p.returncode != 0:
raise BuildException("Error removing sudo for %s:%s" %
(app.id, build.versionName), p.output)
log_path = os.path.join(log_dir, log_path = os.path.join(log_dir,
common.get_toolsversion_logname(app, build)) common.get_toolsversion_logname(app, build))
with open(log_path, 'w') as f: with open(log_path, 'w') as f:

View file

@ -748,6 +748,8 @@ def read_metadata(xref=True, check_vcs=[], sort_by_time=False):
metadatafiles = sorted(metadatafiles) metadatafiles = sorted(metadatafiles)
for metadatapath in metadatafiles: for metadatapath in metadatafiles:
if metadatapath == '.fdroid.txt':
warn_or_exception(_('.fdroid.txt is not supported! Convert to .fdroid.yml or .fdroid.json.'))
packageName, _ignored = fdroidserver.common.get_extension(os.path.basename(metadatapath)) packageName, _ignored = fdroidserver.common.get_extension(os.path.basename(metadatapath))
if packageName in apps: if packageName in apps:
warn_or_exception(_("Found multiple metadata files for {appid}") warn_or_exception(_("Found multiple metadata files for {appid}")

View file

@ -561,7 +561,7 @@ def main():
'-o PasswordAuthentication=no')) '-o PasswordAuthentication=no'))
# TODO vagrant 1.5+ provides `vagrant rsync` # TODO vagrant 1.5+ provides `vagrant rsync`
run_via_vagrant_ssh(v, ['cd ~ && test -d', d, '|| mkdir -p', d]) run_via_vagrant_ssh(v, ['cd ~ && test -d', d, '|| mkdir -p', d])
subprocess.call(['rsync', '-axv', '--progress', '--delete', '-e', subprocess.call(['rsync', '-ax', '--delete', '-e',
ssh_command, ssh_command,
fullpath + '/', fullpath + '/',
user + '@' + hostname + ':~/' + d + '/']) user + '@' + hostname + ':~/' + d + '/'])