[Webkit-unassigned] [Bug 155576] New: Improve update-webkit-dependency script
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Mar 16 19:46:26 PDT 2016
https://bugs.webkit.org/show_bug.cgi?id=155576
Bug ID: 155576
Summary: Improve update-webkit-dependency script
Classification: Unclassified
Product: WebKit
Version: WebKit Local Build
Hardware: PC
OS: Windows 10
Status: NEW
Severity: Normal
Priority: P2
Component: Tools / Tests
Assignee: webkit-unassigned at lists.webkit.org
Reporter: WebKit at JeremyZerfas.com
CC: lforschler at apple.com
The update-webkit-dependency script could use several improvements.
First, every time it is ran it tries to download the entire dependency just in order to get the Last-Modified header so that it can check to see if the dependency is up to date. This could be improved by instead doing a request for just the first byte of the dependency if the server supports range requests and also setting a size limit for the request so the entire dependency is not downloaded in order to get the headers. This can save a decent amount of time and bandwidth for people.
Second, it tries to use the Last-Modified header to determine whether the dependency is up to date and if the server doesn't emit Last-Modified headers (like Dropbox's server which is currently used to store the WinCairoRequirements.zip file and also GitHub as well) it then tries to get the Last-Modified header from a separate *.headers file that is also placed on the server. This could be improved by also using ETag headers (which Dropbox and GitHub both support) to see if the dependency is up to date. When using Dropbox this would generally eliminate the need to maintain those separate *.headers file which seems to cause problems from time to time (see https://lists.webkit.org/pipermail/webkit-help/2015-November/003987.html and bug 153876).
Third, it would also be nice if before installing a dependency, it first checked the zip file to make sure it contains the expected directories. This would help in preventing problems like those in bug 153876.
The patch that I'll be attaching in a moment makes these improvements to the script plus makes a few other minor improvements (more error checking and better error messages mainly).
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160317/bda9a819/attachment.html>
More information about the webkit-unassigned
mailing list