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