<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[198638] trunk/Tools</title>
</head>
<body>

<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt;  }
#msg dl a { font-weight: bold}
#msg dl a:link    { color:#fc3; }
#msg dl a:active  { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff  {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/198638">198638</a></dd>
<dt>Author</dt> <dd>commit-queue@webkit.org</dd>
<dt>Date</dt> <dd>2016-03-24 11:42:57 -0700 (Thu, 24 Mar 2016)</dd>
</dl>

<h3>Log Message</h3>
<pre>Improve update-webkit-dependency script.
https://bugs.webkit.org/show_bug.cgi?id=155576

Patch by Jeremy Zerfas &lt;WebKit@JeremyZerfas.com&gt; on 2016-03-24
Reviewed by Brent Fulgham.

* Scripts/update-webkit-dependency:
-Updated script to also use ETags for helping to determine whether dependency is up to date.
-Updated script so it usually won't need to download the entire dependency each time it is
 ran.
-Updated script to ensure zip file contains expected directories before trying to install.
-Added some more error checking and improved error messages.
(lastModifiedToUnixTime): Deleted.

* Scripts/update-webkit-wincairo-libs:
-Corrected comment about what script does.</pre>

<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkToolsChangeLog">trunk/Tools/ChangeLog</a></li>
<li><a href="#trunkToolsScriptsupdatewebkitdependency">trunk/Tools/Scripts/update-webkit-dependency</a></li>
<li><a href="#trunkToolsScriptsupdatewebkitwincairolibs">trunk/Tools/Scripts/update-webkit-wincairo-libs</a></li>
</ul>

</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkToolsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Tools/ChangeLog (198637 => 198638)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/ChangeLog        2016-03-24 18:42:04 UTC (rev 198637)
+++ trunk/Tools/ChangeLog        2016-03-24 18:42:57 UTC (rev 198638)
</span><span class="lines">@@ -1,3 +1,21 @@
</span><ins>+2016-03-24  Jeremy Zerfas  &lt;WebKit@JeremyZerfas.com&gt;
+
+        Improve update-webkit-dependency script.
+        https://bugs.webkit.org/show_bug.cgi?id=155576
+
+        Reviewed by Brent Fulgham.
+
+        * Scripts/update-webkit-dependency:
+        -Updated script to also use ETags for helping to determine whether dependency is up to date.
+        -Updated script so it usually won't need to download the entire dependency each time it is
+         ran.
+        -Updated script to ensure zip file contains expected directories before trying to install.
+        -Added some more error checking and improved error messages.
+        (lastModifiedToUnixTime): Deleted.
+
+        * Scripts/update-webkit-wincairo-libs:
+        -Corrected comment about what script does.
+
</ins><span class="cx"> 2016-03-24 Bill Ming &lt;mbbill@gmail.com&gt;
</span><span class="cx"> 
</span><span class="cx">         Fix webkitpy tests after r198617
</span></span></pre></div>
<a id="trunkToolsScriptsupdatewebkitdependency"></a>
<div class="modfile"><h4>Modified: trunk/Tools/Scripts/update-webkit-dependency (198637 => 198638)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/Scripts/update-webkit-dependency        2016-03-24 18:42:04 UTC (rev 198637)
+++ trunk/Tools/Scripts/update-webkit-dependency        2016-03-24 18:42:57 UTC (rev 198638)
</span><span class="lines">@@ -2,19 +2,20 @@
</span><span class="cx"> 
</span><span class="cx"> # Copyright (C) 2005, 2006, 2007 Apple Inc.  All rights reserved.
</span><span class="cx"> # Copyright (C) 2011 Carl Lobo.  All rights reserved.
</span><ins>+# Copyright (C) 2016 Jeremy Zerfas.  All rights reserved.
</ins><span class="cx"> #
</span><span class="cx"> # Redistribution and use in source and binary forms, with or without
</span><span class="cx"> # modification, are permitted provided that the following conditions
</span><span class="cx"> # are met:
</span><span class="cx"> #
</span><span class="cx"> # 1.  Redistributions of source code must retain the above copyright
</span><del>-#     notice, this list of conditions and the following disclaimer. 
</del><ins>+#     notice, this list of conditions and the following disclaimer.
</ins><span class="cx"> # 2.  Redistributions in binary form must reproduce the above copyright
</span><span class="cx"> #     notice, this list of conditions and the following disclaimer in the
</span><del>-#     documentation and/or other materials provided with the distribution. 
</del><ins>+#     documentation and/or other materials provided with the distribution.
</ins><span class="cx"> # 3.  Neither the name of Apple Inc. (&quot;Apple&quot;) nor the names of
</span><span class="cx"> #     its contributors may be used to endorse or promote products derived
</span><del>-#     from this software without specific prior written permission. 
</del><ins>+#     from this software without specific prior written permission.
</ins><span class="cx"> #
</span><span class="cx"> # THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS &quot;AS IS&quot; AND ANY
</span><span class="cx"> # EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
</span><span class="lines">@@ -27,7 +28,7 @@
</span><span class="cx"> # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
</span><span class="cx"> # THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
</span><span class="cx"> 
</span><del>-# Updates a development environment to the new WebKitAuxiliaryLibrary
</del><ins>+# Updates a dependency for the development environment.
</ins><span class="cx"> 
</span><span class="cx"> use strict;
</span><span class="cx"> use warnings;
</span><span class="lines">@@ -51,13 +52,12 @@
</span><span class="cx"> Usage:
</span><span class="cx">         update-webkit-dependancy &lt;URL with the dependancy zip file&gt; &lt;*prefix dir inside zip without filename&gt;
</span><span class="cx"> 
</span><del>-        * If filename is requirements.zip and the contents of the zipfile are &quot;requirements/x&quot; then prefix = &quot;.&quot;
-        * If filename is xyz.zip and the contents of the zipfile are xyz/abc/x&quot; then prefix = &quot;abc&quot;
</del><ins>+        * If filename is requirements.zip and the contents of the zip file are &quot;requirements/x&quot; then prefix = &quot;.&quot;
+        * If filename is xyz.zip and the contents of the zip file are xyz/abc/x&quot; then prefix = &quot;abc&quot;
</ins><span class="cx">         * x is lib or include or bin.
</span><span class="cx"> EOF
</span><span class="cx"> }
</span><span class="cx"> 
</span><del>-sub lastModifiedToUnixTime($);
</del><span class="cx"> sub getLibraryName($);
</span><span class="cx"> 
</span><span class="cx"> # Time in seconds that the new zip file must be newer than the old for us to
</span><span class="lines">@@ -71,76 +71,142 @@
</span><span class="cx"> my $prefixInZip = shift;
</span><span class="cx"> my $sourceDir = sourceDir();
</span><span class="cx"> my $file = getLibraryName($libsURL);
</span><del>-my $zipFile = &quot;$file.zip&quot;; 
</del><ins>+my $zipFile = &quot;$file.zip&quot;;
</ins><span class="cx"> my $webkitLibrariesDir = $ENV{'WEBKIT_LIBRARIES'} || File::Spec-&gt;catdir($sourceDir, &quot;WebKitLibraries&quot;, &quot;win&quot;);
</span><span class="cx"> my $tmpRelativeDir = File::Temp::tempdir(&quot;webkitlibsXXXXXXX&quot;, TMPDIR =&gt; 1, CLEANUP =&gt; 1);
</span><span class="cx"> my $tmpAbsDir = File::Spec-&gt;rel2abs($tmpRelativeDir);
</span><span class="cx"> my $ua = LWP::UserAgent-&gt;new();
</span><span class="cx"> $ua-&gt;env_proxy;
</span><span class="cx"> 
</span><del>-print &quot;Checking Last-Modified date of $zipFile...\n&quot;;
</del><ins>+print &quot;Checking for newer version of $zipFile...\n&quot;;
</ins><span class="cx"> 
</span><del>-my $response = $ua-&gt;get($libsURL);
</del><ins>+# Ideally we could just use a previous modification time and/or ETag to do a single conditional request for the file,
+# however there are some problems with this approach. Some servers may not support conditional requests (Dropbox and
+# GitHub don't currently support conditional requests using If-Modified-Since headers plus they also don't emit
+# Last-Modified headers either). If mirrors are being used, some servers may have different ETags for the same file.
+# Some servers may not implement conditional requests properly. Etc...
+#
+# Instead we will assume that we need to download a new version of the file unless we can be reasonably certain that we
+# already previously got an up to date copy of the file. We can do this by making a quick, small request to check for
+# the presence of Last-Modified and/or ETag headers (or alternatively a separate *.headers file that contains a
+# Last-Modified header) and see if they match values that we may have gotten previously.
</ins><span class="cx"> 
</span><del>-unless ($response-&gt;is_success) {
-    print &quot;Could not access $libsURL:\n&quot; . $response-&gt;headers_as_string . &quot;\n&quot;;
-    print &quot;You may not be connected to the internet. Attempting to build without updating.\n&quot;;
-    exit 0;
-}
</del><ins>+# It would be nice to be able to just use a HEAD request for this initial check but developer.apple.com returns 401
+# errors for those requests. Instead we can just set a size limit for a GET request (which we can also set the range for
+# to request just the first byte) so that we don't have to retrieve the entire file.
+$ua-&gt;max_size(0);
+my $response = $ua-&gt;get($libsURL, &quot;Range&quot;, &quot;bytes=0-0&quot;);
+$ua-&gt;max_size(undef);
</ins><span class="cx"> 
</span><del>-my $content_type = $response-&gt;header('Content-Type');
-my $document_length = $response-&gt;header('Content-Length');
-my $modified_time = str2time($response-&gt;header('Last-Modified')); 
</del><ins>+if (! $response-&gt;is_success) {
+    print STDERR &quot;Could not access $libsURL:\n&quot;, $response-&gt;headers_as_string, &quot;\n&quot;;
+    print STDERR &quot;Please ensure that $libsURL is reachable&quot;;
+    if ($libsURL =~ /^https/i) {
+        print STDERR &quot; and that Perl can use LWP::Simple to connect to HTTPS URLs.\n&quot;;
+        print STDERR &quot;You may have to run \&quot;\$ cpan LWP::Protocol::https\&quot;&quot;;
+    }
+    print STDERR &quot;.\n&quot;;
</ins><span class="cx"> 
</span><del>-if (defined $modified_time) {
-    print STDERR &quot;Located a file of type $content_type, of size $document_length.\n&quot;;
-    open NEW, &quot;&gt;&quot;, File::Spec-&gt;catfile($tmpAbsDir, &quot;$file.headers&quot;);
-    print NEW &quot;Last-Modified: &quot; . time2str($modified_time) . &quot;\n&quot;;
-    close NEW;
-} else {
-    #Note: Neither GitHub nor DropBox emit the Last-Modified HTTP header, so fall back to a file
-        #containing the necessary information if we do not receive the information in our initial query.
-    my $headerURL = $libsURL;
-    $headerURL =~ s/\.zip$/\.headers/;
</del><ins>+    if (-f File::Spec-&gt;catfile($webkitLibrariesDir, &quot;$file.headers&quot;)) {
+        print STDERR &quot;Falling back to existing version of $file.\n&quot;;
+        exit 0;
+    } else {
+        print STDERR &quot;No existing version of $file to fall back to.\n&quot;;
+        exit 1;
+    }
+}
</ins><span class="cx"> 
</span><del>-    my $result = getstore($headerURL, File::Spec-&gt;catfile($tmpAbsDir, &quot;$file.headers&quot;));
</del><ins>+my $contentType = $response-&gt;header('Content-Type');
+# Try to determine the file size based on the Content-Range or Content-Length headers. Normally we will need to use the
+# Content-Range header since we are just requesting a range but if the server doesn't support ranges then we will try
+# using the Content-Length header instead. Also note that neither header is affected by using max_size() to limit the
+# response size and the Content-Length header also shouldn't be affected from the use of HTTP compression since LWP
+# doesn't use it by default.
+my $contentLength = $response-&gt;header('Content-Length');
+# If the Content-Range header is defined and has a number after the slash, then that's the file size we want.
+if (defined $response-&gt;header('Content-Range') &amp;&amp; $response-&gt;header('Content-Range') =~ /^.+\/(\d+)/) {
+    $contentLength=$1;
+}
+my $lastModified = $response-&gt;header('Last-Modified');
+my $etag = $response-&gt;header('ETag');
</ins><span class="cx"> 
</span><del>-    if (!is_success($result)) {
-        print STDERR &quot;Couldn't check Last-Modified date of new $zipFile.\n&quot;;
-        print STDERR &quot;Response was: $result.\n&quot;;
-        print STDERR &quot;Please ensure that Perl can use LWP::Simple to connect to HTTPS urls, and that $libsURL is reachable.\n&quot;;
-        print STDERR &quot;You may have to run \$ cpan LWP::Protocol::https\n&quot;;
</del><ins>+print &quot;Located a file&quot;;
+print &quot; of type $contentType&quot; if defined $contentType;
+print &quot; of size $contentLength&quot; if defined $contentLength;
+print &quot;.\n&quot;;
</ins><span class="cx"> 
</span><del>-        if (! -f File::Spec-&gt;catfile($webkitLibrariesDir, &quot;$file.headers&quot;)) {
-            print STDERR &quot;Unable to check Last-Modified date and no version of $file to fall back to.\n&quot;;
-            exit 1;
-        }
</del><ins>+# Get any old headers that are available.
+my $oldLastModified;
+my $oldETag;
+if (open OLDHEADERSFILE, &quot;&lt;&quot;, File::Spec-&gt;catfile($webkitLibrariesDir, &quot;$file.headers&quot;)) {
+    local $/ = undef;
+    my $oldHeaders = &lt;OLDHEADERSFILE&gt;;
+    close OLDHEADERSFILE;
</ins><span class="cx"> 
</span><del>-        print STDERR &quot;Falling back to existing version of $file.\n&quot;;
-        exit 0;
-    }
</del><ins>+    ($oldLastModified) = $oldHeaders =~ /^Last-Modified: (.+)$/mi;
+    ($oldETag) = $oldHeaders =~ /^ETag: (.+)$/mi;
</ins><span class="cx"> }
</span><span class="cx"> 
</span><del>-if (open NEW, File::Spec-&gt;catfile($tmpAbsDir, &quot;$file.headers&quot;)) {
-    my $new = lastModifiedToUnixTime(&lt;NEW&gt;);
-    close NEW;
</del><ins>+# If we have matching old and new ETags, then that is a very good indication that we already had gotten an up to date
+# copy of the file previously. On the other hand if we have ETags that don't match, that doesn't necessarily indicate
+# that the files are different since different servers may compute different ETags for the same file. If they don't
+# match we will continue to see if we can skip downloading the file via other tests.
+if (defined $etag &amp;&amp; defined $oldETag &amp;&amp; $etag eq $oldETag) {
+    print &quot;Current $file is up to date.\n&quot;;
+    exit 0;
+}
</ins><span class="cx"> 
</span><del>-    if (defined $new &amp;&amp; open OLD, File::Spec-&gt;catfile($webkitLibrariesDir, &quot;$file.headers&quot;)) {
-        my $old = lastModifiedToUnixTime(&lt;OLD&gt;);
-        close OLD;
-        if (defined $old &amp;&amp; abs($new - $old) &lt; $newnessThreshold) {
-            print &quot;Current $file is up to date\n&quot;;
-            exit 0;
</del><ins>+# If on the initial request we didn't get a Last-Modified header, then we will try getting the Last-Modified header from
+# a separate *.headers file that may have been put on the server.
+if (! defined $lastModified) {
+    my $headerURL = $libsURL;
+    $headerURL =~ s/\.zip$/\.headers/;
+
+    $response = $ua-&gt;get($headerURL);
+
+    if (! $response-&gt;is_success) {
+        print STDERR &quot;Could not access $headerURL:\n&quot;, $response-&gt;headers_as_string, &quot;\n&quot;;
+        print STDERR &quot;Please ensure that $headerURL is reachable&quot;;
+        if ($headerURL =~ /^https/i) {
+            print STDERR &quot; and that Perl can use LWP::Simple to connect to HTTPS URLs.\n&quot;;
+            print STDERR &quot;You may have to run \&quot;\$ cpan LWP::Protocol::https\&quot;&quot;;
</ins><span class="cx">         }
</span><ins>+        print STDERR &quot;.\n&quot;;
</ins><span class="cx">     }
</span><ins>+
+    ($lastModified) = $response-&gt;content =~ /^Last-Modified: (.+)$/mi;
</ins><span class="cx"> }
</span><span class="cx"> 
</span><ins>+my $lastModifiedTime = str2time($lastModified);
+my $oldLastModifiedTime = str2time($oldLastModified);
+if (defined $lastModifiedTime &amp;&amp; defined $oldLastModifiedTime
+    &amp;&amp; abs($lastModifiedTime-$oldLastModifiedTime) &lt; $newnessThreshold) {
+    print &quot;Current $file is up to date.\n&quot;;
+    exit 0;
+}
+
</ins><span class="cx"> print &quot;Downloading $zipFile...\n\n&quot;;
</span><span class="cx"> print &quot;$libsURL\n&quot;;
</span><span class="cx"> my $result = getstore($libsURL, File::Spec-&gt;catfile($tmpAbsDir, $zipFile));
</span><span class="cx"> die &quot;Couldn't download $zipFile!&quot; if is_error($result);
</span><span class="cx"> 
</span><span class="cx"> my $zip = Archive::Zip-&gt;new(File::Spec-&gt;catfile($tmpAbsDir, $zipFile));
</span><ins>+
+# Make sure the zip file contains a directory with the same name as the zip file (minus the extension) and a prefix
+# directory if applicable.
+my $prefixDirectoryPathInZipFile = &quot;$file/&quot;.(($prefixInZip eq &quot;.&quot;) ? &quot;&quot; : &quot;$prefixInZip/&quot;);
+if (! $zip-&gt;memberNamed($prefixDirectoryPathInZipFile)) {
+    print STDERR &quot;Couldn't find $prefixDirectoryPathInZipFile directory in zip file.\n&quot;;
+
+    if (-f File::Spec-&gt;catfile($webkitLibrariesDir, &quot;$file.headers&quot;)) {
+        print STDERR &quot;Falling back to existing version of $file.\n&quot;;
+        exit 0;
+    } else {
+        print STDERR &quot;No existing version of $file to fall back to.\n&quot;;
+        exit 1;
+    }
+}
+
</ins><span class="cx"> $result = $zip-&gt;extractTree(&quot;&quot;, $tmpAbsDir);
</span><span class="cx"> die &quot;Couldn't unzip $zipFile.&quot; if $result != AZ_OK;
</span><span class="cx"> 
</span><span class="lines">@@ -161,24 +227,20 @@
</span><span class="cx"> 
</span><span class="cx"> File::Find::find(\&amp;wanted, File::Spec-&gt;catfile($tmpAbsDir, $file));
</span><span class="cx"> 
</span><del>-$result = move(File::Spec-&gt;catfile($tmpAbsDir, &quot;$file.headers&quot;), $webkitLibrariesDir);
-print STDERR &quot;Couldn't move $file.headers to $webkitLibrariesDir&quot; . &quot;.\n&quot; if $result == 0;
</del><ins>+if (open HEADERSFILE, &quot;&gt;&quot;, File::Spec-&gt;catfile($webkitLibrariesDir, &quot;$file.headers&quot;)) {
+    print HEADERSFILE &quot;Last-Modified: $lastModified\n&quot; if defined $lastModified;
+    print HEADERSFILE &quot;ETag: $etag\n&quot; if defined $etag;
+    close HEADERSFILE;
+} else {
+    print STDERR &quot;Couldn't write $file.headers to $webkitLibrariesDir.\n&quot;
+}
</ins><span class="cx"> 
</span><span class="cx"> print &quot;The $file has been sucessfully installed in\n $webkitLibrariesDir\n&quot;;
</span><span class="cx"> exit;
</span><span class="cx"> 
</span><del>-sub lastModifiedToUnixTime($)
-{
-    my ($str) = @_;
-
-    $str =~ /^Last-Modified: (.*)$/ or return;
-    return str2time($1);
-}
-
</del><span class="cx"> sub getLibraryName($)
</span><span class="cx"> {
</span><span class="cx">     my $url = shift;
</span><span class="cx">     $url =~ m#/([^/]+)\.zip$#;
</span><span class="cx">     return $1;
</span><span class="cx"> }
</span><del>-
</del></span></pre></div>
<a id="trunkToolsScriptsupdatewebkitwincairolibs"></a>
<div class="modfile"><h4>Modified: trunk/Tools/Scripts/update-webkit-wincairo-libs (198637 => 198638)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Tools/Scripts/update-webkit-wincairo-libs        2016-03-24 18:42:04 UTC (rev 198637)
+++ trunk/Tools/Scripts/update-webkit-wincairo-libs        2016-03-24 18:42:57 UTC (rev 198638)
</span><span class="lines">@@ -8,13 +8,13 @@
</span><span class="cx"> # are met:
</span><span class="cx"> #
</span><span class="cx"> # 1.  Redistributions of source code must retain the above copyright
</span><del>-#     notice, this list of conditions and the following disclaimer. 
</del><ins>+#     notice, this list of conditions and the following disclaimer.
</ins><span class="cx"> # 2.  Redistributions in binary form must reproduce the above copyright
</span><span class="cx"> #     notice, this list of conditions and the following disclaimer in the
</span><del>-#     documentation and/or other materials provided with the distribution. 
</del><ins>+#     documentation and/or other materials provided with the distribution.
</ins><span class="cx"> # 3.  Neither the name of Apple Inc. (&quot;Apple&quot;) nor the names of
</span><span class="cx"> #     its contributors may be used to endorse or promote products derived
</span><del>-#     from this software without specific prior written permission. 
</del><ins>+#     from this software without specific prior written permission.
</ins><span class="cx"> #
</span><span class="cx"> # THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS &quot;AS IS&quot; AND ANY
</span><span class="cx"> # EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
</span><span class="lines">@@ -27,7 +27,7 @@
</span><span class="cx"> # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
</span><span class="cx"> # THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
</span><span class="cx"> 
</span><del>-# Updates a development environment to the new WebKitAuxiliaryLibrary
</del><ins>+# Updates a development environment to the new WinCairoRequirements.
</ins><span class="cx"> 
</span><span class="cx"> use strict;
</span><span class="cx"> use warnings;
</span></span></pre>
</div>
</div>

</body>
</html>