[webkit-reviews] review granted: [Bug 51790] [WinCairo] Patch to download the WinCairo dependencies as part of build-webkit. : [Attachment 89269] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 12 14:19:00 PDT 2011


Adam Roben (:aroben) <aroben at apple.com> has granted Brent Fulgham
<bfulgham at webkit.org>'s request for review:
Bug 51790: [WinCairo] Patch to download the WinCairo dependencies as part of
build-webkit.
https://bugs.webkit.org/show_bug.cgi?id=51790

Attachment 89269: Patch
https://bugs.webkit.org/attachment.cgi?id=89269&action=review

------- Additional Comments from Adam Roben (:aroben) <aroben at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=89269&action=review

> Tools/Scripts/build-webkit:512
> +    if(isWinCairo()) {

Missing space after "if".

> Tools/Scripts/update-webkit-dependency:56
> +sub getLibraryName;

You should add the prototype here, too.

> Tools/Scripts/update-webkit-dependency:154
> +    my $path = shift;
> +    my ($volume, $directory, $file) = File::Spec->splitpath($path);

Hm, splitpath doesn't seem right, since $path is really a URL. You should
rename $path to $url, and use the regex solution I think.


More information about the webkit-reviews mailing list