Bring up queues for iOS 12 (Build fix)
authorjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 21 Sep 2018 17:13:34 +0000 (17:13 +0000)
committerjbedard@apple.com <jbedard@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 21 Sep 2018 17:13:34 +0000 (17:13 +0000)
https://bugs.webkit.org/show_bug.cgi?id=189683

Unreviewed build fix.

We should not build ImageDiff as x86 by default since 32 bit projects are
deprecated in Xcode. ImageDiff should only be built once.

* Scripts/build-webkit:
* Scripts/webkitdirs.pm:
(argumentsForConfiguration):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@236331 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/build-webkit
Tools/Scripts/webkitdirs.pm

index 7381dac..da78450 100644 (file)
@@ -1,3 +1,17 @@
+2018-09-21  Jonathan Bedard  <jbedard@apple.com>
+
+        Bring up queues for iOS 12 (Build fix)
+        https://bugs.webkit.org/show_bug.cgi?id=189683
+
+        Unreviewed build fix.
+
+        We should not build ImageDiff as x86 by default since 32 bit projects are
+        deprecated in Xcode. ImageDiff should only be built once.
+
+        * Scripts/build-webkit:
+        * Scripts/webkitdirs.pm:
+        (argumentsForConfiguration):
+
 2018-09-21  Adrian Perez de Castro  <aperez@igalia.com>
 
         [WPE] Built RPM of WPE webkit fails to install with "nothing provides libWPEToolingBackends.so()"
index dcdbaa5..e008a2c 100755 (executable)
@@ -344,24 +344,24 @@ if (isAppleWinWebKit() || isWinCairo()) {
         if (exitStatus($result)) {
             exit exitStatus($result);
         }
+    }
 
-        # Build ImageDiff for host
-        my @command = File::Spec->catfile(getcwd(), "/Tools/Scripts/build-imagediff");
-        chdirWebKit();
-        if (!-e $command[0]) {
-          die "build-imagediff script not found";
-        }
+    # Build ImageDiff for host
+    my @command = File::Spec->catfile(getcwd(), "/Tools/Scripts/build-imagediff");
+    chdirWebKit();
+    if (!-e $command[0]) {
+        die "build-imagediff script not found";
+    }
 
-        if ($clean) {
-          push @command, " --clean";
-        }
-        push @command, argumentsForConfiguration();
-        push @command, @ARGV;
-        @command = extractNonMacOSHostConfiguration(\@command);
-        $result = system(@command);
-        if (exitStatus($result)) {
-            exit exitStatus($result);
-        }
+    if ($clean) {
+        push @command, " --clean";
+    }
+    push @command, argumentsForConfiguration();
+    push @command, @ARGV;
+    @command = extractNonMacOSHostConfiguration(\@command);
+    $result = system(@command);
+    if (exitStatus($result)) {
+        exit exitStatus($result);
     }
 }
 
index c5a66da..beebbc3 100755 (executable)
@@ -488,7 +488,7 @@ sub argumentsForConfiguration()
     push(@args, '--release') if ($configuration =~ "^Release");
     push(@args, '--ios-device') if (defined $xcodeSDK && $xcodeSDK =~ /^iphoneos/);
     push(@args, '--ios-simulator') if (defined $xcodeSDK && $xcodeSDK =~ /^iphonesimulator/);
-    push(@args, '--32-bit') if ($architecture ne "x86_64" and !isWin64());
+    push(@args, '--32-bit') if ($architecture eq "x86" and !isWin64());
     push(@args, '--64-bit') if (isWin64());
     push(@args, '--gtk') if isGtk();
     push(@args, '--wpe') if isWPE();