Francois Gouget : testbot/WineRunReconfig: Notify the administrator if the build fails.

[Alexandre Julliard]

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

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Tue Sep 24 11:00:40 2019 +0200

testbot/WineRunReconfig: Notify the administrator if the build fails.

Build failures prevent the TestBot snapshot from being updated
(leading to an out-of-date Wine and patches that fail to apply)
and can typically only be fixed manually.

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

---

 testbot/bin/WineRunReconfig.pl | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/testbot/bin/WineRunReconfig.pl b/testbot/bin/WineRunReconfig.pl
index fe36ad0..063bab2 100755
--- a/testbot/bin/WineRunReconfig.pl
+++ b/testbot/bin/WineRunReconfig.pl
@@ -438,6 +438,11 @@ if ($TA->GetFile("Reconfig.log", "$TaskDir/log"))
   {
     # We should not have badpatch errors and if the result line is missing we
     # probably already have an error message that explains why.
+    NotifyAdministrator("The ". $VM->Name ." build failed",
+                        "The ". $VM->Name ." build failed:\n\n".
+                        "$Summary->{Task}\n\n".
+                        "See the link below for more details:\n".
+                        MakeSecureURL(GetTaskURL($JobId, $StepNo, $TaskNo)) ."\n");
     $NewStatus = "badbuild";
   }
 }