A secondary test was added to check for harfbuzz-icu when it was split but the test wasn't properly quoted. If the test fails you get an error about missing version "./configure.9.8" instead of the expected "0.9.8".
Created attachment 205701 [details] Patch
The patch looks OK, but requires a ChangeLog entry. The contributing documentation has some guidelines on how to prepare the ChangeLog. http://www.webkit.org/coding/contributing.html
Created attachment 205703 [details] Patch
(In reply to comment #2) > The patch looks OK, but requires a ChangeLog entry. Yes I had some issues with webkit-patch, so I manually created a diff with a ChangeLog entry and uploaded directly.
Thanks for adding the changelog entry, but could you explain in the changelog what's the problem you are fixing? something similar to the first comment in this bug.
Created attachment 205705 [details] Patch
(In reply to comment #5) > but could you explain in the changelog what's the problem you are fixing? I've included more detail in the most recent attachment.
Comment on attachment 205705 [details] Patch Thanks! It seems there are more cases like this one in FindDependencies.m4 we should fix them also eventually.
Comment on attachment 205705 [details] Patch Clearing flags on attachment: 205705 Committed r152163: <http://trac.webkit.org/changeset/152163>
All reviewed patches have been landed. Closing bug.