[PATCH] testbot: Add some section headers to the scripts.

Francois Gouget fgouget at codeweavers.com
Wed Jun 13 04:45:45 CDT 2018


Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/bin/Janitor.pl         |  8 ++++++++
 testbot/bin/WineRunBuild.pl    |  4 ++++
 testbot/bin/WineRunReconfig.pl |  4 ++++
 testbot/bin/WineRunTask.pl     |  9 +++++++++
 testbot/bin/WineSendLog.pl     | 10 ++++++++++
 5 files changed, 35 insertions(+)

diff --git a/testbot/bin/Janitor.pl b/testbot/bin/Janitor.pl
index f7a6e33b6..9fb38efde 100755
--- a/testbot/bin/Janitor.pl
+++ b/testbot/bin/Janitor.pl
@@ -53,6 +53,11 @@ use WineTestBot::Tasks;
 use WineTestBot::Users;
 use WineTestBot::VMs;
 
+
+#
+# Logging and error handling helpers
+#
+
 my $LogOnly;
 sub Trace(@)
 {
@@ -67,6 +72,9 @@ sub Error(@)
 }
 
 
+#
+# Setup and command line processing
+#
 
 $ENV{PATH} = "/usr/bin:/bin";
 delete $ENV{ENV};
diff --git a/testbot/bin/WineRunBuild.pl b/testbot/bin/WineRunBuild.pl
index 61deb50a6..31decc107 100755
--- a/testbot/bin/WineRunBuild.pl
+++ b/testbot/bin/WineRunBuild.pl
@@ -49,6 +49,10 @@ use WineTestBot::Log;
 use WineTestBot::Engine::Notify;
 
 
+#
+# Logging and error handling helpers
+#
+
 my $Debug;
 sub Debug(@)
 {
diff --git a/testbot/bin/WineRunReconfig.pl b/testbot/bin/WineRunReconfig.pl
index 713712cec..99b73da6f 100755
--- a/testbot/bin/WineRunReconfig.pl
+++ b/testbot/bin/WineRunReconfig.pl
@@ -48,6 +48,10 @@ use WineTestBot::Log;
 use WineTestBot::Engine::Notify;
 
 
+#
+# Logging and error handling helpers
+#
+
 my $Debug;
 sub Debug(@)
 {
diff --git a/testbot/bin/WineRunTask.pl b/testbot/bin/WineRunTask.pl
index d75741757..3290b0bc6 100755
--- a/testbot/bin/WineRunTask.pl
+++ b/testbot/bin/WineRunTask.pl
@@ -47,6 +47,10 @@ use WineTestBot::Log;
 use WineTestBot::Engine::Notify;
 
 
+#
+# Logging and error handling helpers
+#
+
 my $Debug;
 sub Debug(@)
 {
@@ -60,6 +64,11 @@ sub Error(@)
   LogMsg @_;
 }
 
+
+#
+# Task helpers
+#
+
 sub TakeScreenshot($$)
 {
   my ($VM, $FullScreenshotFileName) = @_;
diff --git a/testbot/bin/WineSendLog.pl b/testbot/bin/WineSendLog.pl
index 2a8942701..e03395af1 100755
--- a/testbot/bin/WineSendLog.pl
+++ b/testbot/bin/WineSendLog.pl
@@ -501,6 +501,11 @@ EOF
   }
 }
 
+
+#
+# Setup and command line processing
+#
+
 $ENV{PATH} = "/usr/bin:/bin";
 delete $ENV{ENV};
 
@@ -528,6 +533,11 @@ if (! defined($Job))
   exit(1);
 }
 
+
+#
+# Analyze the log and notify the developer
+#
+
 SendLog($Job);
 
 LogMsg "Log for job $JobId sent\n";
-- 
2.17.1



More information about the wine-devel mailing list