chiark / gitweb /
build: include buildserverid on wiki build logs, if it exists
authorHans-Christoph Steiner <hans@eds.org>
Wed, 3 Jan 2018 12:58:06 +0000 (13:58 +0100)
committerHans-Christoph Steiner <hans@eds.org>
Wed, 3 Jan 2018 12:58:06 +0000 (13:58 +0100)
When `fdroid build` is run using the buildserver, it should fetch the
buildserverid on the first build.

Seems this was really a silly bug in 837fc99d74f7694a64f014f3d38a8b07d9e8b3bd

fdroidserver/build.py

index 8bbe8aab75d68bb82e97877c94e4e9b0a47bbd36..83cb18ab1e559caa7a8d7142fc8d56385364db9e 100644 (file)
@@ -79,6 +79,7 @@ def build_server(app, build, vcs, build_dir, output_dir, log_dir, force):
             buildserverid = subprocess.check_output(['vagrant', 'ssh', '-c',
                                                      'cat /home/vagrant/buildserverid'],
                                                     cwd='builder').rstrip()
+            logging.debug(_('Fetched buildserverid from VM: ') + buildserverid)
 
         # Open SSH connection...
         logging.info("Connecting to virtual machine...")
@@ -1306,7 +1307,7 @@ def main():
                           + "* this build completed at " + _get_build_timestamp() + '\n' \
                           + '* fdroidserverid: [https://gitlab.com/fdroid/fdroidserver/commit/' \
                           + fdroidserverid + ' ' + fdroidserverid + ']\n\n'
-                    if options.onserver:
+                    if buildserverid:
                         txt += '* buildserverid: [https://gitlab.com/fdroid/fdroidserver/commit/' \
                                + buildserverid + ' ' + buildserverid + ']\n\n'
                     txt += tools_version_log + '\n\n'