Added skipped mbed library build reporting and removed unnecessary print

workspace_tools/build_release.py

@@ -108,6 +108,7 @@ if __name__ == '__main__':
     start = time()
     failures = []
     successes = []
+    skips = []
     build_report = []
     for target_name, toolchain_list in OFFICIAL_MBED_LIBRARY_BUILD:
         if options.official_only:
@@ -121,14 +122,21 @@ if __name__ == '__main__':
             toolchains = toolchainSet and set((options.toolchains).split(','))
 
 
-        cur_target_build_report = { "target": target_name, "passing": [], "failing": []}
+        cur_target_build_report = { "target": target_name, "passing": [], "failing": [], "skipped": []}
 
         for toolchain in toolchains:
             id = "%s::%s" % (target_name, toolchain)
             try:
-                build_mbed_libs(TARGET_MAP[target_name], toolchain, verbose=options.verbose, jobs=options.jobs)
+                built_mbed_lib = build_mbed_libs(TARGET_MAP[target_name], toolchain, verbose=options.verbose, jobs=options.jobs)
+
+                if built_mbed_lib:
                     successes.append(id)
                     cur_target_build_report["passing"].append({ "toolchain": toolchain })
+                else:
+                    skips.append(id)
+                    cur_target_build_report["skipped"].append({ "toolchain": toolchain })
+
+
             except Exception, e:
                 failures.append(id)
                 cur_target_build_report["failing"].append({ "toolchain": toolchain })
@@ -148,6 +156,10 @@ if __name__ == '__main__':
         print "\n\nBuild successes:"
         print "\n".join(["  * %s" % s for s in successes])
 
+    if skips:
+        print "\n\nBuild skips:"
+        print "\n".join(["  * %s" % s for s in skips])
+
     if failures:
         print "\n\nBuild failures:"
         print "\n".join(["  * %s" % f for f in failures])

workspace_tools/ci_templates/library_build/build_report.html

@@ -8,7 +8,7 @@
       <span class="greenbold">[PASS]</span>
       {% endif %}
 
-      {{report.target}} - Passing: {{report.passing|length}}, Failing: {{report.failing|length}}
+      {{report.target}} - Passing: {{report.passing|length}}, Failing: {{report.failing|length}}, Skipped: {{report.skipped|length}}
     </a>
   </h3>
 
@@ -22,5 +22,10 @@
     {% with build = report.passing %}
     {% include 'library_build/build_report_table.html' %}
     {% endwith %}
+
+    <h4>Skipped</h4>
+    {% with build = report.skipped %}
+    {% include 'library_build/build_report_table.html' %}
+    {% endwith %}
   </div>
 </div>

workspace_tools/ci_templates/tests_build/build_report.html

@@ -8,7 +8,7 @@
       <span class="greenbold">[PASS]</span>
       {% endif %}
 
-      {{report.target}} - Passing: {{report.passing|length}}, Failing: {{report.failing|length}}
+      {{report.target}} - Passing: {{report.passing|length}}, Failing: {{report.failing|length}}, Skipped: {{report.skipped|length}}
     </a>
   </h3>
 
@@ -22,5 +22,10 @@
     {% with build = report.passing %}
     {% include 'tests_build/build_report_table.html' %}
     {% endwith %}
+
+    <h4>Skipped</h4>
+    {% with build = report.skipped %}
+    {% include 'tests_build/build_report_table.html' %}
+    {% endwith %}
   </div>
 </div>

workspace_tools/test_api.py

@@ -455,7 +455,6 @@ class SingleTestRunner(object):
 
                 except ToolException:
                     project_name_str = project_name if project_name is not None else test_id
-                    print "DIS BE MAH ERRRRR: %s" % (str(ToolException))
                     print self.logger.log_line(self.logger.LogType.ERROR, 'There were errors while building project %s'% (project_name_str))
                     build_report[toolchain]["test_build_failing"].append(test_id)
                     # return self.test_summary, self.shuffle_random_seed, self.test_summary_ext, self.test_suite_properties_ext