Summary: | Fix the EFL and GTK build after r182243 | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> | ||||
Component: | New Bugs | Assignee: | Zan Dobersek <zan> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Blocker | CC: | achristensen, commit-queue, fpizlo, ossy, zan | ||||
Priority: | P1 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Csaba Osztrogonác
2015-04-02 23:16:51 PDT
I'll produce something. Created attachment 250058 [details]
Patch
Attachment 250058 [details] did not pass style-queue: ERROR: Source/WebKit2/PlatformGTK.cmake:425: There should be exactly one empty line instead of 0 between "" <file alias=\"images/panIcon\">panIcon.png</file>\n"" and "" <file alias=\"images/textAreaResizeCorner\">textAreaResizeCorner.png</file>\n"". [list/emptyline] [5] ERROR: Source/WebKit2/PlatformGTK.cmake:426: Alphabetical sorting problem. "" <file alias=\"images/textAreaResizeCorner@2x\">textAreaResizeCorner@2x.png</file>\n"" should be before "" <file alias=\"images/textAreaResizeCorner\">textAreaResizeCorner.png</file>\n"". [list/order] [5] Total errors found: 2 in 6 files If any of these errors are false positives, please file a bug against check-webkit-style. (In reply to comment #2) > Created attachment 250058 [details] > Patch Fixes clean builds for the GTK port. EFL fix is speculative, so please test it out. (In reply to comment #4) > (In reply to comment #2) > > Created attachment 250058 [details] > > Patch > > Fixes clean builds for the GTK port. EFL fix is speculative, so please test > it out. Thanks for the fix, I'll check it immediately. Comment on attachment 250058 [details]
Patch
It works on EFL too, so r=me.
Comment on attachment 250058 [details] Patch Clearing flags on attachment: 250058 Committed r182315: <http://trac.webkit.org/changeset/182315> All reviewed patches have been landed. Closing bug. Comment on attachment 250058 [details]
Patch
Thank you for the fix !
|