Francois Gouget : testbot: Add a note regarding the detection of new errors.

Alexandre Julliard julliard at winehq.org
Wed Aug 29 15:37:53 CDT 2018


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Sun Aug 26 10:17:47 2018 +0200

testbot: Add a note regarding the detection of new errors.

The detection of new errors may be based on a WineTest report which is
not up-to-date. However this should not be a real issue in practice.

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

---

 testbot/bin/WineSendLog.pl | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/testbot/bin/WineSendLog.pl b/testbot/bin/WineSendLog.pl
index f3b56d8..b0efa8c 100755
--- a/testbot/bin/WineSendLog.pl
+++ b/testbot/bin/WineSendLog.pl
@@ -266,6 +266,12 @@ EOF
   # Build a job summary with only the new errors
   #
 
+  # Note that this may be a bit inaccurate right after a Wine commit if this
+  # job's patch got compiled on top of the new Wine before all the reference
+  # WineTest results were updated. This is made more likely by the job
+  # priorities: high for Wine updates, and low for WineTest runs.
+  # However in practice this would only be an issue if the patch reintroduced
+  # an error that just disappeared in the latest Wine which is highly unlikely.
   my @Messages;
   foreach my $Key (@SortedKeys)
   {




More information about the wine-cvs mailing list