Use isAnyWindows() instead of isCygwin() || isWindows() in Perl scripts.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Jan 2016 18:41:46 +0000 (18:41 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 28 Jan 2016 18:41:46 +0000 (18:41 +0000)
https://bugs.webkit.org/show_bug.cgi?id=153594

Patch by Konstantin Tokarev <annulen@yandex.ru> on 2016-01-28
Reviewed by Alex Christensen.

* Scripts/webkitdirs.pm:
(determineNumberOfCPUs):
(jscPath):
(checkFrameworks):
(setupCygwinEnv):
(wrapperPrefixIfNeeded):
(cmakeGeneratedBuildfile):
(generateBuildSystemFromCMakeProject):
* Scripts/webkitperl/features.pm:
(libraryContainsSymbol):

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

Tools/ChangeLog
Tools/Scripts/webkitdirs.pm
Tools/Scripts/webkitperl/features.pm

index 7e6f75b..7c7263e 100644 (file)
@@ -1,3 +1,21 @@
+2016-01-28  Konstantin Tokarev  <annulen@yandex.ru>
+
+        Use isAnyWindows() instead of isCygwin() || isWindows() in Perl scripts.
+        https://bugs.webkit.org/show_bug.cgi?id=153594
+
+        Reviewed by Alex Christensen.
+
+        * Scripts/webkitdirs.pm:
+        (determineNumberOfCPUs):
+        (jscPath):
+        (checkFrameworks):
+        (setupCygwinEnv):
+        (wrapperPrefixIfNeeded):
+        (cmakeGeneratedBuildfile):
+        (generateBuildSystemFromCMakeProject):
+        * Scripts/webkitperl/features.pm:
+        (libraryContainsSymbol):
+
 2016-01-28  Chris Dumez  <cdumez@apple.com>
 
         Unreviewed, rolling out r195742.
index 3c15a65..d4a29d7 100755 (executable)
@@ -396,7 +396,7 @@ sub determineNumberOfCPUs
         if ($numberOfCPUs eq "") {
             $numberOfCPUs = (grep /processor/, `cat /proc/cpuinfo`);
         }
-    } elsif (isWindows() || isCygwin()) {
+    } elsif (isAnyWindows()) {
         # Assumes cygwin
         $numberOfCPUs = `ls /proc/registry/HKEY_LOCAL_MACHINE/HARDWARE/DESCRIPTION/System/CentralProcessor | wc -w`;
     } elsif (isDarwin() || isBSD()) {
@@ -417,7 +417,7 @@ sub jscPath($)
     my ($productDir) = @_;
     my $jscName = "jsc";
     $jscName .= "_debug"  if configuration() eq "Debug_All";
-    $jscName .= ".exe" if (isWindows() || isCygwin());
+    $jscName .= ".exe" if (isAnyWindows());
     return "$productDir/$jscName" if -e "$productDir/$jscName";
     return "$productDir/JavaScriptCore.framework/Resources/$jscName";
 }
@@ -929,7 +929,7 @@ sub builtDylibPathForName
 # Check to see that all the frameworks are built.
 sub checkFrameworks # FIXME: This is a poor name since only the Mac calls built WebCore a Framework.
 {
-    return if isCygwin() || isWindows();
+    return if isAnyWindows();
     my @frameworks = ("JavaScriptCore", "WebCore");
     push(@frameworks, "WebKit") if isAppleMacWebKit(); # FIXME: This seems wrong, all ports should have a WebKit these days.
     for my $framework (@frameworks) {
@@ -1641,7 +1641,7 @@ sub setupAppleWinEnv()
 
 sub setupCygwinEnv()
 {
-    return if !isCygwin() && !isWindows();
+    return if !isAnyWindows();
     return if $vcBuildPath;
 
     my $programFilesPath = programFilesPath();
@@ -1810,7 +1810,7 @@ sub isCachedArgumentfileOutOfDate($@)
 
 sub wrapperPrefixIfNeeded()
 {
-    if (isWindows() || isCygwin()) {
+    if (isAnyWindows()) {
         return ();
     }
     if (isAppleMacWebKit()) {
@@ -1919,7 +1919,7 @@ sub cmakeGeneratedBuildfile(@)
     my ($willUseNinja) = @_;
     if ($willUseNinja) {
         return File::Spec->catfile(baseProductDir(), configuration(), "build.ninja")
-    } elsif (isWindows() || isCygwin()) {
+    } elsif (isAnyWindows()) {
         return File::Spec->catfile(baseProductDir(), configuration(), "WebKit.sln")
     } else {
         return File::Spec->catfile(baseProductDir(), configuration(), "Makefile")
@@ -1977,7 +1977,7 @@ sub generateBuildSystemFromCMakeProject
     # Compiler options to keep floating point values consistent
     # between 32-bit and 64-bit architectures.
     determineArchitecture();
-    if ($architecture ne "x86_64" && !isARM() && !isCrossCompilation() && !isWindows() && !isCygwin()) {
+    if ($architecture ne "x86_64" && !isARM() && !isCrossCompilation() && !isAnyWindows()) {
         $ENV{'CXXFLAGS'} = "-march=pentium4 -msse2 -mfpmath=sse " . ($ENV{'CXXFLAGS'} || "");
     }
 
index a0057cd..abe03bd 100644 (file)
@@ -49,7 +49,7 @@ sub libraryContainsSymbol($$)
 {
     my ($path, $symbol) = @_;
 
-    if (isCygwin() or isWindows()) {
+    if (isAnyWindows()) {
         # FIXME: Implement this for Windows.
         return 0;
     }