harfbuzz requires ragel package to be installed.
Created attachment 199640 [details] Patch
Comment on attachment 199640 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=199640&action=review Maybe correct the name in the ChangeLog? A good way to learn about annoying reviewers ;) > Tools/ChangeLog:9 > + which is expected when building hardbuff Not very important, but the name is "harfbuzz".
Informally reviewing... There is another small typo in the ChangeLog, please change “scrtip” to “script”. It would be nice to fix it when you edit the ChangeLog with Xan's comment. Apart from that, looks fine.
Created attachment 199644 [details] Patch
Comment on attachment 199644 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=199644&action=review > Tools/ChangeLog:9 > + which is expected when building HarfBuff. Still wrong :D! See http://www.freedesktop.org/wiki/Software/HarfBuzz
Created attachment 199650 [details] Patch
Comment on attachment 199650 [details] Patch Clearing flags on attachment: 199650 Committed r149109: <http://trac.webkit.org/changeset/149109>
All reviewed patches have been landed. Closing bug.