Francois Gouget : testbot/WineRunReconfig: Clarify the email in case of a failed build.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: c94f7a70fdfbf416a46f4ab053d0e12c554ffc78
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=c94f7a70fdfbf416a46f4ab0...

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Wed Feb 26 10:37:41 2020 +0100

testbot/WineRunReconfig: Clarify the email in case of a failed build.

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 testbot/bin/WineRunReconfig.pl | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/testbot/bin/WineRunReconfig.pl b/testbot/bin/WineRunReconfig.pl
index 897b675..644d546 100755
--- a/testbot/bin/WineRunReconfig.pl
+++ b/testbot/bin/WineRunReconfig.pl
@@ -436,9 +436,11 @@ if ($TA->GetFile("Reconfig.log", "$TaskDir/task.log"))
   {
     # We should not have badpatch errors and if the result line is missing we
     # probably already have an error message that explains why.
+    my $Status = $LogInfo->{Task} eq "missing" ?
+        "the log contains no status line." :
+        "the last log status line says $LogInfo->{Task}.";
     NotifyAdministrator("The ". $VM->Name ." build failed",
-                        "The ". $VM->Name ." build failed:\n\n".
-                        "$LogInfo->{Task}\n\n".
+                        "The ". $VM->Name ." build failed and $Status\n\n".
                         "See the link below for more details:\n".
                         MakeSecureURL(GetTaskURL($JobId, $StepNo, $TaskNo)) ."\n");
     $NewStatus = "badbuild";