Feature flags not present for Visual Studio when using build-webkit
authordon.olmstead@sony.com <don.olmstead@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Nov 2018 22:10:30 +0000 (22:10 +0000)
committerdon.olmstead@sony.com <don.olmstead@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Nov 2018 22:10:30 +0000 (22:10 +0000)
https://bugs.webkit.org/show_bug.cgi?id=191288

Reviewed by Fujii Hironori.

Passes feature flags to Visual studio builds within build-webkit.
Removes cmakeBasedPortArguments as it is no longer in use.

* Scripts/build-jsc:
* Scripts/build-webkit:
* Scripts/webkitdirs.pm:
(cmakeBasedPortArguments): Deleted.

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

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

index 8f13301..d09bd4c 100644 (file)
@@ -1,3 +1,18 @@
+2018-11-06  Don Olmstead  <don.olmstead@sony.com>
+
+        Feature flags not present for Visual Studio when using build-webkit
+        https://bugs.webkit.org/show_bug.cgi?id=191288
+
+        Reviewed by Fujii Hironori.
+
+        Passes feature flags to Visual studio builds within build-webkit.
+        Removes cmakeBasedPortArguments as it is no longer in use.
+
+        * Scripts/build-jsc:
+        * Scripts/build-webkit:
+        * Scripts/webkitdirs.pm:
+        (cmakeBasedPortArguments): Deleted.
+
 2018-11-06  Per Arne Vollan  <pvollan@apple.com>
 
         REGRESSION (r230523): Caps lock indicator not shown in password field
index fe60f21..57a443b 100755 (executable)
@@ -162,7 +162,7 @@ if (isCMakeBuild()) {
     }
 
     # This call only returns if nothing wrong happened
-    buildCMakeProjectOrExit(0, undef, $buildTarget, (cmakeBasedPortArguments(), @featureArgs, @cmakeArgs));
+    buildCMakeProjectOrExit(0, undef, $buildTarget, @featureArgs, @cmakeArgs);
     writeCongrats();
     exit exitStatus(0);
 }
index 8560c16..8239674 100755 (executable)
@@ -291,7 +291,7 @@ if (isCMakeBuild() && !isAnyWindows()) {
     my @featureArgs = cmakeArgsFromFeatures(@features, !$noExperimentalFeatures);
     removeCMakeCache(@featureArgs);
 
-    buildCMakeProjectOrExit($clean, $prefixPath, $makeArgs, (cmakeBasedPortArguments(), @featureArgs), @cmakeArgs);
+    buildCMakeProjectOrExit($clean, $prefixPath, $makeArgs, @featureArgs, @cmakeArgs);
 }
 
 my $baseProductDir = baseProductDir();
@@ -300,7 +300,7 @@ if (isAppleWinWebKit() || isWinCairo() || isPlayStation()) {
     removeCMakeCache(@featureArgs);
 
     chdirWebKit();
-    if (exitStatus(generateBuildSystemFromCMakeProject($prefixPath, @cmakeArgs))) {
+    if (exitStatus(generateBuildSystemFromCMakeProject($prefixPath, @featureArgs, @cmakeArgs))) {
         die "Run Visual Studio 2017 installation vcvars.bat before build-webkit when using ninja";
     }
 
index 30a5388..2102adb 100755 (executable)
@@ -66,7 +66,6 @@ BEGIN {
        &chdirWebKit
        &checkFrameworks
        &cmakeArgsFromFeatures
-       &cmakeBasedPortArguments
        &currentSVNRevision
        &debugSafari
        &executableProductDir
@@ -2367,12 +2366,6 @@ sub cmakeArgsFromFeatures(\@;$)
     return @args;
 }
 
-
-sub cmakeBasedPortArguments()
-{
-    return ();
-}
-
 sub cmakeBasedPortName()
 {
     return ucfirst portName();