testbot/WineSendLog: Remove FatalError().
Francois Gouget
fgouget at codeweavers.com
Thu May 1 07:22:59 CDT 2014
Its main point was to trace the JobId but the $Job parameter was never
used. So the code is clearer without it.
---
testbot/bin/WineSendLog.pl | 18 +++++-------------
1 file changed, 5 insertions(+), 13 deletions(-)
diff --git a/testbot/bin/WineSendLog.pl b/testbot/bin/WineSendLog.pl
index b375447..24cdd48 100755
--- a/testbot/bin/WineSendLog.pl
+++ b/testbot/bin/WineSendLog.pl
@@ -42,16 +42,6 @@ use WineTestBot::Jobs;
use WineTestBot::Log;
use WineTestBot::StepsTasks;
-sub FatalError
-{
- my ($ErrMessage, $Job) = @_;
-
- my $JobKey = defined($Job) ? $Job->GetKey() : "0";
-
- LogMsg "$JobKey $ErrMessage";
-
- exit 1;
-}
sub IsBotFailure
{
@@ -535,17 +525,19 @@ if ($JobId =~ /^(\d+)$/)
}
else
{
- FatalError "Invalid JobId $JobId\n";
+ LogMsg "Invalid JobId $JobId\n";
+ exit(1);
}
my $Job = CreateJobs()->GetItem($JobId);
if (! defined($Job))
{
- FatalError "Job $JobId doesn't exist\n";
+ LogMsg "Job $JobId doesn't exist\n";
+ exit(1);
}
SendLog($Job);
LogMsg "Log for job $JobId sent\n";
-exit;
+exit(0);
--
1.9.2
More information about the wine-patches
mailing list