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

[mojca at macports.org]

Revision: 151675
          https://trac.macports.org/changeset/151675
Author:   mojca at macports.org
Date:     2016-08-19 14:31:03 -0700 (Fri, 19 Aug 2016)
Log Message:
-----------
buildbot-test: slight optimization

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

Modified: contrib/buildbot-test/master.cfg
===================================================================
--- contrib/buildbot-test/master.cfg	2016-08-19 21:23:20 UTC (rev 151674)
+++ contrib/buildbot-test/master.cfg	2016-08-19 21:31:03 UTC (rev 151675)
@@ -519,7 +519,7 @@
         text.append("\nLog from failed builds:")
         summary_step = [x for x in build.getSteps() if x.getName() == "summary"][0]
         summary_log  = [x for x in summary_step.getLogs() if x.getName() == "stdio"][0]
-        failed_ports_dict = {}
+        failed_ports_set = set()
         pattern = re.compile("^Building '(?P<port>.*?)'.*?(\(failed to install dependency '(?P<dependency>.*?)'\))?$")
         # iterate through all the ports being built
         for line in summary_log.getText().splitlines():
@@ -529,11 +529,10 @@
                 match = pattern.match(line)
                 if match:
                     for key in ['port', 'dependency']:
-                        if key in match.groupdict():
-                            port = match.groupdict()[key]
-                            if port:
-                                failed_ports_dict[port] = True
-        failed_ports = sorted(failed_ports_dict.keys())
+                        port = match.groupdict().get(key):
+                        if port:
+                            failed_ports_set.add(port)
+        failed_ports = sorted(failed_ports_set)
         text.append("\nBroken ports:\n\t- {:s}".format("\n\t- ".join(failed_ports)))
 
         maintainers_to_notify = list()
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-changes/attachments/20160819/09dae9d0/attachment.html>