Francois Gouget : testbot: Rebuild TestLauncher as part of the Reconfig.

Alexandre Julliard julliard at winehq.org
Mon Jun 12 12:34:33 CDT 2017


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Mon Jun 12 18:26:45 2017 +0200

testbot: Rebuild TestLauncher as part of the Reconfig.

Currently the TestLauncher binaries are checked into Git which is wrong
and means they are effectively never updated. Now that Reconfig.pl rebuilds
them the TestBot server will be able to retrive the up-to-date binaries
and use them.
Modify the TestLauncher makefile so it does not overwrite the checked-in
TestLauncher binaries.

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

---

 testbot/bin/build/Reconfig.pl     | 30 +++++++++++++++++++-----------
 testbot/src/TestLauncher/Makefile |  9 ++++++---
 2 files changed, 25 insertions(+), 14 deletions(-)

diff --git a/testbot/bin/build/Reconfig.pl b/testbot/bin/build/Reconfig.pl
index e522e94..dfc338b 100755
--- a/testbot/bin/build/Reconfig.pl
+++ b/testbot/bin/build/Reconfig.pl
@@ -115,6 +115,20 @@ sub BuildTestAgentd()
   return 1;
 }
 
+sub BuildTestLauncher()
+{
+  system("( cd $::RootDir/src/TestLauncher && set -x && " .
+         "  time make -j$ncpus" .
+         ") >>$LogDir/Reconfig.log 2>&1");
+  if ($? != 0)
+  {
+    LogMsg "Build TestLauncher failed\n";
+    return !1;
+  }
+
+  return 1;
+}
+
 sub BuildNative()
 {
   mkdir "$DataDir/build-native" if (! -d "$DataDir/build-native");
@@ -173,17 +187,11 @@ if (! GitPull())
 
 CountCPUs();
 
-if (! BuildTestAgentd())
-{
-  exit(1);
-}
-
-if (! BuildNative())
-{
-  exit(1);
-}
-
-if (! BuildCross(32) || ! BuildCross(64))
+if (!BuildTestAgentd() ||
+    !BuildTestLauncher() ||
+    !BuildNative() ||
+    !BuildCross(32) ||
+    !BuildCross(64))
 {
   exit(1);
 }
diff --git a/testbot/src/TestLauncher/Makefile b/testbot/src/TestLauncher/Makefile
index 2bce2a1..06b3729 100644
--- a/testbot/src/TestLauncher/Makefile
+++ b/testbot/src/TestLauncher/Makefile
@@ -5,12 +5,15 @@ CROSSSTRIP32 = i686-w64-mingw32-strip
 CROSSCC64    = x86_64-w64-mingw32-gcc
 CROSSSTRIP64 = x86_64-w64-mingw32-strip
 
-all: $(bindir)/TestLauncher32.exe $(bindir)/TestLauncher64.exe
+all: TestLauncher32.exe TestLauncher64.exe
 
-$(bindir)/TestLauncher32.exe: TestLauncher.c
+TestLauncher32.exe: TestLauncher.c
 	$(CROSSCC32) -Wall -o $@ $<
 	$(CROSSSTRIP32) $@
 
-$(bindir)/TestLauncher64.exe: TestLauncher.c
+TestLauncher64.exe: TestLauncher.c
 	$(CROSSCC64) -Wall -o $@ $<
 	$(CROSSSTRIP64) $@
+
+clean:
+	rm -f TestLauncher32.exe TestLauncher64.exe




More information about the wine-cvs mailing list