From: Zhang Qiang <qiang.z.zhang@intel.com>
Date: Thu, 28 Nov 2013 05:20:51 +0000 (+0800)
Subject: Adjust output of final report
X-Git-Tag: 0.11~11
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d5f7879a2054417e1563044fa1f66229297f4e97;p=tools%2Fdepanneur.git

Adjust output of final report

- always print dirs of rpm/srpm/build roots, even if some packages failed
- print build log dir

Change-Id: I55ecfe70880753b90d61e5a67fbd0de07fdd571c
Fixes: #1466
---

diff --git a/depanneur b/depanneur
index 5975feb..ce25410 100755
--- a/depanneur
+++ b/depanneur
@@ -1522,7 +1522,6 @@ sub build_html_report
     open(my $report_html, '>', "$localrepo/$dist/$arch/index.html");
     $tmpl->output(print_to => $report_html);
     close($report_html);
-    info("generated html format report:\n     $localrepo/$dist/$arch/index.html" );
 }
 
 
@@ -1535,7 +1534,7 @@ sub build_json_report
 
 sub build_report
 {
-    my $msg = "*** Error Summary ***\n";
+    my $msg = "*** Build Status Summary ***\n";
 
     my $total_packages = scalar(keys %to_build) - scalar (@skipped) + scalar (@export_errors);
     my $succeeded_packages = scalar(keys %succeeded);
@@ -1614,13 +1613,17 @@ sub build_report
     build_html_report();
     build_json_report();
 
+    info($msg);
+
+    info("generated html format report:\n     $localrepo/$dist/$arch/index.html" );
+    info("generated RPM packages can be found from local repo:\n     $rpm_repo_path");
+    info("generated source RPM packages can be found from local repo:\n     $srpm_repo_path");
+    info("build logs can be found in:\n     $localrepo/$dist/$arch/logs");
+    info("build roots located in:\n     $scratch_dir.*");
     if (%errors || %expansion_errors || @export_errors) {
-        error($msg);
-    } else {
-        info("generated RPM packages can be found from local repo:\n     $rpm_repo_path");
-        info("generated source RPM packages can be found from local repo:\n     $srpm_repo_path");
-        info("build roots located in:\n     $scratch_dir.*");
+        exit 1;
     }
+
 }
 
 sub get_binary_list() {