Francois Gouget : testbot/build: Update the source before the native build!.

Alexandre Julliard julliard at winehq.org
Thu Apr 19 16:47:20 CDT 2018


Module: tools
Branch: master
Commit: 4d0543ea645a16111f8f175de4732914a1034e57
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=4d0543ea645a16111f8f175de4732914a1034e57

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Thu Apr 19 03:32:34 2018 +0200

testbot/build: Update the source before the native build!.

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

---

 testbot/bin/build/Build.pl | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/testbot/bin/build/Build.pl b/testbot/bin/build/Build.pl
index 94f87e2..19f220b 100755
--- a/testbot/bin/build/Build.pl
+++ b/testbot/bin/build/Build.pl
@@ -138,16 +138,7 @@ sub ApplyPatch($$$)
   if ($? != 0)
   {
     LogMsg "Patch failed to apply\n";
-    return (-1, $NeedMakeInclude, $NeedBuildDeps, $NeedImplib, $NeedConfigure);
-  }
-
-  if ($NeedBuildNative)
-  {
-    InfoMsg "Building tools\n";
-    if (!BuildNative())
-    {
-      return (-1, $NeedMakeInclude, $NeedBuildDeps, $NeedImplib, $NeedConfigure);
-    }
+    return (-1, $NeedMakeInclude, $NeedBuildDeps, $NeedImplib, $NeedConfigure, $NeedBuildNative);
   }
 
   if ($NeedMakeMakefiles)
@@ -159,7 +150,7 @@ sub ApplyPatch($$$)
     if ($? != 0)
     {
       LogMsg "make_makefiles failed\n";
-      return (-1, $NeedMakeInclude, $NeedBuildDeps, $NeedImplib, $NeedConfigure);
+      return (-1, $NeedMakeInclude, $NeedBuildDeps, $NeedImplib, $NeedConfigure, $NeedBuildNative);
     }
   }
 
@@ -173,7 +164,7 @@ sub ApplyPatch($$$)
     {
        LogMsg "Autoconf failed\n";
        return (-1, $NeedMakeInclude, $NeedBuildDeps, $NeedImplib,
-               $NeedConfigure);
+               $NeedConfigure, $NeedBuildNative);
     }
     $NeedConfigure = 1;
   }
@@ -193,7 +184,7 @@ sub ApplyPatch($$$)
   }
 
   return ($NeedMakefile, $NeedMakeInclude, $NeedBuildDeps, $NeedImplib,
-          $NeedConfigure);
+          $NeedConfigure, $NeedBuildNative);
 }
 
 my $ncpus;
@@ -418,7 +409,7 @@ else
 }
 
 my ($NeedMakefile, $NeedMakeInclude, $NeedBuildDeps, $NeedImplib,
-    $NeedConfigure) = ApplyPatch($PatchFile, $PatchType, $BaseName);
+    $NeedConfigure, $NeedBuildNative) = ApplyPatch($PatchFile, $PatchType, $BaseName);
 if ($NeedMakefile < 0)
 {
   exit(1);
@@ -426,6 +417,15 @@ if ($NeedMakefile < 0)
 
 CountCPUs();
 
+if ($NeedBuildNative)
+{
+  InfoMsg "Building tools\n";
+  if (!BuildNative())
+  {
+    exit(1);
+  }
+}
+
 if ($Run32 && ! BuildTestExecutable($BaseName, $PatchType, 32,
                                     $NeedConfigure, 0 < $NeedMakefile,
                                     $NeedMakeInclude, $NeedBuildDeps,




More information about the wine-cvs mailing list