[154097] contrib/buildbot-test/master.cfg

[larryv at macports.org]

Revision: 154097
          https://trac.macports.org/changeset/154097
Author:   larryv at macports.org
Date:     2016-10-19 20:06:20 -0700 (Wed, 19 Oct 2016)
Log Message:
-----------
buildbot: Ensure subject exists when portWatcherMessageFormatter returns

Modified Paths:
--------------
    contrib/buildbot-test/master.cfg

Modified: contrib/buildbot-test/master.cfg
===================================================================
--- contrib/buildbot-test/master.cfg	2016-10-20 03:00:57 UTC (rev 154096)
+++ contrib/buildbot-test/master.cfg	2016-10-20 03:06:20 UTC (rev 154097)
@@ -704,6 +704,7 @@
 
 def portWatcherMessageFormatter(mode, name, build, results, master_status, interested_users):
     result = util.Results[results]
+    subject = 'Build {:s}'.format(result.title())
     text = list()
     text.append('Status:       {:s}'.format(result.title()))
     text.append('Build slave:  {:s}'.format(build.getSlavename()))
@@ -759,7 +760,6 @@
             text.append('- {:s}\n  {:s}'.format(k, build_urls_dict[k]))
         text.append('\n-- \nBest regards,\nMacPorts Buildbot\n{:s}'.format(c['buildbotURL']))
 
-        subject = 'Build {:s}'.format(result.title())
         if failed_ports:
             subject += ': '
             subject += ', '.join(sorted(failed_ports)[:10])
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macports.org/pipermail/macports-changes/attachments/20161019/0dcaa66e/attachment-0002.html>