Fix the Windows buildbot when WebKitAuxiliaryLibs.zip cannot be found on our server.
authorbweinstein@apple.com <bweinstein@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 Jun 2010 04:58:19 +0000 (04:58 +0000)
committerbweinstein@apple.com <bweinstein@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 Jun 2010 04:58:19 +0000 (04:58 +0000)
Reviewed by Darin Adler.

Robustify update-webkit-auxiliary-libs to be able to handle the case when WebKitAuxiliaryLibs.zip doesn't exist,
and allow the script to fall back to an existing file, only failing if there is no fallback file. Also switch to checking
WEXITSTATUS on $result instead of just comparing $result to 0.

* Scripts/update-webkit-auxiliary-libs:

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

WebKitTools/ChangeLog
WebKitTools/Scripts/update-webkit-auxiliary-libs

index 70f8289..93abf66 100644 (file)
@@ -1,3 +1,15 @@
+2010-06-10  Brian Weinstein  <bweinstein@apple.com>
+
+        Reviewed by Darin Adler.
+
+        Fix the Windows buildbot when WebKitAuxiliaryLibs.zip cannot be found on our server.
+        
+        Robustify update-webkit-auxiliary-libs to be able to handle the case when WebKitAuxiliaryLibs.zip doesn't exist,
+        and allow the script to fall back to an existing file, only failing if there is no fallback file. Also switch to checking
+        WEXITSTATUS on $result instead of just comparing $result to 0.
+
+        * Scripts/update-webkit-auxiliary-libs:
+
 2010-06-10  Ojan Vafai  <ojan@chromium.org>
 
         Reviewed by Alexey Proskuryakov.
index 9c52449..19e4ad3 100755 (executable)
 use strict;
 use warnings;
 
-use HTTP::Date qw(str2time);
 use File::Find;
-use File::Temp ();
 use File::Spec;
+use File::Temp ();
 use FindBin;
+use HTTP::Date qw(str2time);
+use POSIX;
 use lib $FindBin::Bin;
 use webkitdirs;
 
@@ -58,9 +59,21 @@ my $tmpDir = File::Spec->rel2abs(File::Temp::tempdir("webkitlibsXXXXXXX", TMPDIR
 print "Checking Last-Modified date of $zipFile...\n";
 
 my $result = system "curl -s -I $auxiliaryLibsURL | grep Last-Modified > \"$tmpDir/$file.headers\"";
-print STDERR "Couldn't check Last-Modified date of new $zipFile.\n" if $result;
 
-if (!$result && open NEW, "$tmpDir/$file.headers") {
+if (WEXITSTATUS($result)) {
+    print STDERR "Couldn't check Last-Modified date of new $zipFile.\n";
+    print STDERR "Please ensure that $auxiliaryLibsURL is reachable.\n";
+
+    if (! -f "$webkitLibrariesDir/$file.headers") {
+        print STDERR "Unable to check Last-Modified date and no version of $file to fall back to.\n";
+        exit 1;
+    }
+
+    print STDERR "Falling back to existing version of $file.\n";
+    exit 0;
+}
+
+if (open NEW, "$tmpDir/$file.headers") {
     my $new = lastModifiedToUnixTime(<NEW>);
     close NEW;