Francois Gouget : testbot: Fix the "Set doesn't affect tests" message.
Alexandre Julliard
julliard at winehq.org
Thu May 31 14:31:28 CDT 2018
Module: tools
Branch: master
Commit: 004b1384aae737b10ab8cb4d36324d3be7d75dd2
URL: https://source.winehq.org/git/tools.git/?a=commit;h=004b1384aae737b10ab8cb4d36324d3be7d75dd2
Author: Francois Gouget <fgouget at codeweavers.com>
Date: Thu May 31 10:04:32 2018 +0200
testbot: Fix the "Set doesn't affect tests" message.
There is no need to rerun the tests for part N of a patch set if it
does not impact the tests: the results would be the same as those
already obtained from the combined part 1 .. part N patch.
So adjust the message to better convey this.
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
testbot/lib/WineTestBot/Patches.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/testbot/lib/WineTestBot/Patches.pm b/testbot/lib/WineTestBot/Patches.pm
index 71d735b..d5be1b3 100644
--- a/testbot/lib/WineTestBot/Patches.pm
+++ b/testbot/lib/WineTestBot/Patches.pm
@@ -137,7 +137,7 @@ sub Submit($$$)
my $Impacts = GetPatchImpact("$DataDir/patches/" . $self->Id, undef, $PastImpacts);
if (!$Impacts->{UnitCount})
{
- $self->Disposition(($IsSet ? "Set" : "Patch") .
+ $self->Disposition(($IsSet ? "Part" : "Patch") .
" doesn't affect tests");
return undef;
}
More information about the wine-cvs
mailing list