From 15d5fc06fcd66024e32e95b66707bea33522fff8 Mon Sep 17 00:00:00 2001
From: "cltbld%netscape.com" <cltbld%netscape.com>
Date: Mon, 19 Aug 2002 23:26:22 +0000
Subject: [PATCH] Forgot to pass test_result back in run_all_tests, this was
 causing build status to be zero in some cases, which the server was
 rejecting.  -mcafee

---
 tools/tinderbox/build-seamonkey-util.pl | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/tools/tinderbox/build-seamonkey-util.pl b/tools/tinderbox/build-seamonkey-util.pl
index 0728b12984d..662dd2a8d5d 100644
--- a/tools/tinderbox/build-seamonkey-util.pl
+++ b/tools/tinderbox/build-seamonkey-util.pl
@@ -22,7 +22,7 @@ use File::Path;     # for rmtree();
 use Config;         # for $Config{sig_name} and $Config{sig_num}
 use File::Find ();
 
-$::UtilsVersion = '$Revision: 1.179 $ ';
+$::UtilsVersion = '$Revision: 1.180 $ ';
 
 package TinderUtils;
 
@@ -621,8 +621,6 @@ sub BuildIt {
     my $exit_early = 0;
     my $start_time = 0;
 
-    my $status = 0;
-
     my $build_failure_count = 0;  # Keep count of build failures.
 
     # Bypass profile manager at startup.
@@ -754,6 +752,7 @@ sub BuildIt {
             mkdir $Settings::ObjDir, 0777 if ($Settings::ObjDir && ! -e $Settings::ObjDir);
 
             # Run the make command.
+            my $status = 0;
             $status = run_shell_command "$make $targets";
             if ($status != 0) {
                 $build_status = 'busted';
@@ -1590,6 +1589,9 @@ sub run_all_tests {
                                             "-P $Settings::MozProfileName",
                                             "file:$startup_build_dir/../startup-test.html");
     }
+
+    # Done with tests.  Pass test result back.
+    return $test_result;
 }