New public Ewk headers have been added but they were not marked as installable. We need to fix this.
Created attachment 152242 [details] Patch
Comment on attachment 152242 [details] Patch LGTM. Do you really need to specify the absolute paths in the list, BTW?
(In reply to comment #2) > (From update of attachment 152242 [details]) > LGTM. Do you really need to specify the absolute paths in the list, BTW? no clue, I merely followed the existing style.
LGTM.
(In reply to comment #2) > (From update of attachment 152242 [details]) > LGTM. Do you really need to specify the absolute paths in the list, BTW? Or you can define a new variable for this path. But since this is the only place using the particular path, IMO it is not necessary. The patch is fine the way it is now.
(In reply to comment #5) > (In reply to comment #2) > > (From update of attachment 152242 [details] [details]) > > LGTM. Do you really need to specify the absolute paths in the list, BTW? > > Or you can define a new variable for this path. But since this is the only place using the particular path, IMO it is not necessary. The patch is fine the way it is now. What I mean is that it shouldn't be necessary to define an absolute path at all.
Comment on attachment 152242 [details] Patch Clearing flags on attachment: 152242 Committed r122597: <http://trac.webkit.org/changeset/122597>
All reviewed patches have been landed. Closing bug.