[Tools] Remove $additionalCMakeArgs variable because without this variable --cmakearg...
authorjh718.park@samsung.com <jh718.park@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 3 Feb 2016 23:06:52 +0000 (23:06 +0000)
committerjh718.park@samsung.com <jh718.park@samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 3 Feb 2016 23:06:52 +0000 (23:06 +0000)
https://bugs.webkit.org/show_bug.cgi?id=153827

Reviewed by Michael Catanzaro.

When an array is passed to subroutines as argument in perl,
a list containing its element is returned from the array.
Currently @$ arguments are being passed, then it is going to be one list
including all the elements not only from @ but also from $.
As a result, --cmakeargs option's value is included in @cmakeArgs
at generateBuildSystemFromCMakeProject subroutine,
so $additionalCMakeArgs doesn't needed anymore.

* Scripts/webkitdirs.pm:
(generateBuildSystemFromCMakeProject):

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

Tools/ChangeLog
Tools/Scripts/webkitdirs.pm

index 8949d63..46b6684 100644 (file)
@@ -1,3 +1,21 @@
+2016-02-03  Joonghun Park  <jh718.park@samsung.com>
+
+        [Tools] Remove $additionalCMakeArgs variable because without this variable --cmakeargs options works well
+        https://bugs.webkit.org/show_bug.cgi?id=153827
+
+        Reviewed by Michael Catanzaro.
+
+        When an array is passed to subroutines as argument in perl,
+        a list containing its element is returned from the array.
+        Currently @$ arguments are being passed, then it is going to be one list
+        including all the elements not only from @ but also from $.
+        As a result, --cmakeargs option's value is included in @cmakeArgs
+        at generateBuildSystemFromCMakeProject subroutine,
+        so $additionalCMakeArgs doesn't needed anymore.
+
+        * Scripts/webkitdirs.pm:
+        (generateBuildSystemFromCMakeProject):
+
 2016-02-03  Jer Noble  <jer.noble@apple.com>
 
         [Mac] Wrap a resource and resource loader in a NSURLSession-like object for use by lower level frameworks
index fbdd68a..48ea91b 100755 (executable)
@@ -1977,7 +1977,7 @@ sub cmakeGeneratedBuildfile(@)
 
 sub generateBuildSystemFromCMakeProject
 {
-    my ($prefixPath, @cmakeArgs, $additionalCMakeArgs) = @_;
+    my ($prefixPath, @cmakeArgs) = @_;
     my $config = configuration();
     my $port = cmakeBasedPortName();
     my $buildPath = File::Spec->catdir(baseProductDir(), $config);
@@ -2018,7 +2018,6 @@ sub generateBuildSystemFromCMakeProject
     # Don't warn variables which aren't used by cmake ports.
     push @args, "--no-warn-unused-cli";
     push @args, @cmakeArgs if @cmakeArgs;
-    push @args, $additionalCMakeArgs if $additionalCMakeArgs;
 
     my $cmakeSourceDir = isCygwin() ? windowsSourceDir() : sourceDir();
     push @args, '"' . $cmakeSourceDir . '"';