Francois Gouget : testbot/Build: Detect when make_requests needs to be re-run.
Alexandre Julliard
julliard at winehq.org
Wed Oct 17 11:31:30 CDT 2018
Module: tools
Branch: master
Commit: c59a90577cb67892384800329963a3da812ef937
URL: https://source.winehq.org/git/tools.git/?a=commit;h=c59a90577cb67892384800329963a3da812ef937
Author: Francois Gouget <fgouget at codeweavers.com>
Date: Wed Oct 17 16:25:42 2018 +0200
testbot/Build: Detect when make_requests needs to be re-run.
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
testbot/lib/Build/Utils.pm | 11 +++++++++++
testbot/lib/WineTestBot/PatchUtils.pm | 8 +++++++-
2 files changed, 18 insertions(+), 1 deletion(-)
diff --git a/testbot/lib/Build/Utils.pm b/testbot/lib/Build/Utils.pm
index 2e6cc85..aa5c205 100644
--- a/testbot/lib/Build/Utils.pm
+++ b/testbot/lib/Build/Utils.pm
@@ -114,6 +114,17 @@ sub ApplyPatch($$)
}
my $Impacts = GetPatchImpacts($PatchFile);
+ if ($Impacts->{MakeRequests})
+ {
+ InfoMsg "\nRunning make_requests\n";
+ system("cd '$DataDir/$Dir' && set -x && ./tools/make_requests");
+ if ($? != 0)
+ {
+ LogMsg "make_requests failed\n";
+ return undef;
+ }
+ }
+
if ($Impacts->{MakeMakefiles})
{
InfoMsg "\nRunning make_makefiles\n";
diff --git a/testbot/lib/WineTestBot/PatchUtils.pm b/testbot/lib/WineTestBot/PatchUtils.pm
index 502a781..7df50f9 100644
--- a/testbot/lib/WineTestBot/PatchUtils.pm
+++ b/testbot/lib/WineTestBot/PatchUtils.pm
@@ -300,6 +300,11 @@ sub GetPatchImpacts($)
$Impacts->{PatchedRoot} = $Impacts->{MakeMakefiles} = 1;
$Impacts->{IsWinePatch} = 1;
}
+ elsif ($Line =~ m=^--- \w+/server/protocol\.def$=)
+ {
+ $Impacts->{PatchedRoot} = $Impacts->{MakeRequests} = 1;
+ $Impacts->{IsWinePatch} = 1;
+ }
elsif ($Line =~ m=^--- /dev/null$=)
{
$Change = "new";
@@ -344,7 +349,7 @@ sub GetPatchImpacts($)
$Impacts->{BuildModules} = {};
# Also backup the build-related fields.
- foreach my $Field ("Autoconf", "MakeMakefiles",
+ foreach my $Field ("Autoconf", "MakeMakefiles", "MakeRequests",
"PatchedRoot", "PatchedModules", "PatchedTests")
{
$PastImpacts->{$Field} = $Impacts->{$Field};
@@ -435,6 +440,7 @@ sub GetPatchImpacts($)
{
$Impacts->{Autoconf} ||= $PastImpacts->{Autoconf};
$Impacts->{MakeMakefiles} ||= $PastImpacts->{MakeMakefiles};
+ $Impacts->{MakeRequests} ||= $PastImpacts->{MakeRequests};
$Impacts->{RebuildRoot} ||= $PastImpacts->{PatchedRoot};
$Impacts->{RebuildModules} ||= $PastImpacts->{PatchedModules};
map { $Impacts->{BuildModules}->{$_} = 1 } keys %{$PastImpacts->{BuildModules}};
More information about the wine-cvs
mailing list