parallel_emerge: clean up failed summary
When multiple packages fail, the output currently looks like:
Packages failed: chromeos-base/vboot_reference-1.0-r764 ,chromeos-base/power_manager-0.0.1-r622
The spacing around the commas is incorrect, and with a lot of packages,
it is much harder scan. Throw some newlines in there to get:
Packages failed:
chromeos-base/vboot_reference-1.0-r764
chromeos-base/power_manager-0.0.1-r622
BUG=None
TEST=`./build_packages` fails multiple packages and shows nice output
Change-Id: Ibf03439a516adb625d8aaddeb60059e97023c7eb
Reviewed-on: https://gerrit.chromium.org/gerrit/37148
Reviewed-by: Brian Harring <ferringb@chromium.org>
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
diff --git a/scripts/parallel_emerge.py b/scripts/parallel_emerge.py
index 568ae57..4f4d634 100644
--- a/scripts/parallel_emerge.py
+++ b/scripts/parallel_emerge.py
@@ -1443,7 +1443,7 @@
else:
# Tell the user why we're exiting.
if self._failed:
- print "Packages failed: %s" % " ,".join(self._failed)
+ print 'Packages failed:\n\t%s' % '\n\t'.join(self._failed)
status_file = os.environ.get("PARALLEL_EMERGE_STATUS_FILE")
if status_file:
failed_pkgs = set(portage.cpv_getkey(x) for x in self._failed)