Bug 91232 - [EFL][WK2] Make new public Ewk headers as installable
Summary: [EFL][WK2] Make new public Ewk headers as installable
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
Depends on:
Blocks: 61838
  Show dependency treegraph
 
Reported: 2012-07-13 06:29 PDT by Chris Dumez
Modified: 2012-07-13 10:09 PDT (History)
9 users (show)

See Also:


Attachments
Patch (1.66 KB, patch)
2012-07-13 06:30 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2012-07-13 06:29:17 PDT
New public Ewk headers have been added but they were not marked as installable. We need to fix this.
Comment 1 Chris Dumez 2012-07-13 06:30:28 PDT
Created attachment 152242 [details]
Patch
Comment 2 Raphael Kubo da Costa (:rakuco) 2012-07-13 08:06:13 PDT
Comment on attachment 152242 [details]
Patch

LGTM. Do you really need to specify the absolute paths in the list, BTW?
Comment 3 Chris Dumez 2012-07-13 08:08:48 PDT
(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.
Comment 4 Thiago Marcos P. Santos 2012-07-13 08:52:22 PDT
LGTM.
Comment 5 Thiago Marcos P. Santos 2012-07-13 08:55:53 PDT
(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.
Comment 6 Raphael Kubo da Costa (:rakuco) 2012-07-13 09:27:06 PDT
(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 7 WebKit Review Bot 2012-07-13 10:09:14 PDT
Comment on attachment 152242 [details]
Patch

Clearing flags on attachment: 152242

Committed r122597: <http://trac.webkit.org/changeset/122597>
Comment 8 WebKit Review Bot 2012-07-13 10:09:21 PDT
All reviewed patches have been landed.  Closing bug.