<rdar://problem/8930699> build-webkit gives a bogus warning with newer versions of...
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 29 Jan 2011 00:46:59 +0000 (00:46 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 29 Jan 2011 00:46:59 +0000 (00:46 +0000)
Reviewed by Mark Rowe.

* Scripts/webkitdirs.pm:
(checkRequiredSystemConfig): Check the Xcode marketing version
in addition to the DevCoreTools build version before complaining
about an old version of Xcode.  Also make the Mac OS X version
check use Perl's built-in version string comparitor.

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

Tools/ChangeLog
Tools/Scripts/webkitdirs.pm

index ee91c882d328572ba0234e73351c6e95606e92de..7319fb1b3fcfc7d02f1a4d0c1cbc13726f3f1fdd 100644 (file)
@@ -1,3 +1,15 @@
+2011-01-28  David Kilzer  <ddkilzer@apple.com>
+
+        <rdar://problem/8930699> build-webkit gives a bogus warning with newer versions of Xcode 
+
+        Reviewed by Mark Rowe.
+
+        * Scripts/webkitdirs.pm:
+        (checkRequiredSystemConfig): Check the Xcode marketing version
+        in addition to the DevCoreTools build version before complaining
+        about an old version of Xcode.  Also make the Mac OS X version
+        check use Perl's built-in version string comparitor.
+
 2011-01-28  Dirk Pranke  <dpranke@chromium.org>
 
         Unreviewed, build fix.
index 9ee56d3eff0ce4238dd1162ee87781cc6c458479..aa7bab79a96c8369114d56727ca2871b9c381323 100644 (file)
@@ -1009,14 +1009,18 @@ sub checkRequiredSystemConfig
 {
     if (isDarwin()) {
         chomp(my $productVersion = `sw_vers -productVersion`);
-        if ($productVersion lt "10.4") {
+        if (eval "v$productVersion" lt v10.4) {
             print "*************************************************************\n";
             print "Mac OS X Version 10.4.0 or later is required to build WebKit.\n";
             print "You have " . $productVersion . ", thus the build will most likely fail.\n";
             print "*************************************************************\n";
         }
-        my $xcodeVersion = `xcodebuild -version`;
-        if ($xcodeVersion !~ /DevToolsCore-(\d+)/ || $1 < 747) {
+        my $xcodebuildVersionOutput = `xcodebuild -version`;
+        my $devToolsCoreVersion = ($xcodebuildVersionOutput =~ /DevToolsCore-(\d+)/) ? $1 : undef;
+        my $xcodeVersion = ($xcodebuildVersionOutput =~ /Xcode ([0-9](\.[0-9]+)*)/) ? $1 : undef;
+        if (!$devToolsCoreVersion && !$xcodeVersion
+            || $devToolsCoreVersion && $devToolsCoreVersion < 747
+            || $xcodeVersion && eval "v$xcodeVersion" lt v2.3) {
             print "*************************************************************\n";
             print "Xcode Version 2.3 or later is required to build WebKit.\n";
             print "You have an earlier version of Xcode, thus the build will\n";