[tools] testbot/build: Run make depend before rebuilding the tests.

Francois Gouget fgouget at codeweavers.com
Tue May 31 12:15:40 CDT 2022


'make buildtest' only generates the headers needed by the tests. So it's
possible for a patch to introduce a dependency on a header that was not
build by Reconfig.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/bin/build/Build.pl | 4 ++--
 testbot/lib/Build/Utils.pm | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/testbot/bin/build/Build.pl b/testbot/bin/build/Build.pl
index 23ab68587..8502f212d 100755
--- a/testbot/bin/build/Build.pl
+++ b/testbot/bin/build/Build.pl
@@ -167,8 +167,8 @@ if ($DataDir =~ /'/)
 my $Impacts = ApplyPatch("wine", $PatchFile);
 
 if (!$Impacts or
-    !BuildWine($TaskMissions, 0, "exe32", undef, "buildtests") or
-    !BuildWine($TaskMissions, 0, "exe64", undef, "buildtests"))
+    !BuildWine($TaskMissions, 0, "exe32", undef, ["depend", "buildtests"]) or
+    !BuildWine($TaskMissions, 0, "exe64", undef, ["depend", "buildtests"]))
 {
   exit(1);
 }
diff --git a/testbot/lib/Build/Utils.pm b/testbot/lib/Build/Utils.pm
index 9e73cba59..8a84f5e6a 100644
--- a/testbot/lib/Build/Utils.pm
+++ b/testbot/lib/Build/Utils.pm
@@ -330,7 +330,7 @@ our $TRY_WERROR = 2;
 sub BuildWine($$$$;$)
 {
   my ($TaskMissions, $Flags, $Build, $Configure, $Targets) = @_;
-  $Targets ||= "";
+  $Targets ||= [""];
 
   return 1 if (!$TaskMissions->{Builds}->{$Build});
   # Rebuild from scratch to make sure cruft will not accumulate
@@ -344,14 +344,14 @@ sub BuildWine($$$$;$)
   {
     $Rc = system("cd '$DataDir/wine-$Build' && set -x && ".
                  "time ../wine/configure $Configure --enable-werror && ".
-                 "time make -j$CPUCount $Targets");
+                 join(" && ", map { "time make -j$CPUCount $_" } @$Targets));
     LogMsg "With -Werror the $Build Wine build fails\n" if ($Rc);
   }
   if ($Rc)
   {
     $Rc = system("cd '$DataDir/wine-$Build' && set -x && ".
                  (defined $Configure ? "time ../wine/configure $Configure && " : "").
-                 "time make -j$CPUCount $Targets");
+                 join(" && ", map { "time make -j$CPUCount $_" } @$Targets));
   }
   if ($Rc)
   {
-- 
2.30.2




More information about the wine-devel mailing list