Summary: | [CMAKE][WK2] include cmakeconfig.h in WebKit2/config.h | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryuan Choi <ryuan.choi> | ||||||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | gyuyoung.kim, kenneth, leandro, lucas.de.marchi, rakuco, tonikitoo, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 61838 | ||||||||||
Attachments: |
|
Description
Ryuan Choi
2011-06-14 20:08:20 PDT
Created attachment 97241 [details]
Patch
Looks OK to me, informal r+. Comment on attachment 97241 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=97241&action=review > Source/WebKit2/config.h:61 > -#if defined(BUILDING_QT__) || defined(BUILDING_GTK__) > +#if defined(BUILDING_QT__) || defined(BUILDING_GTK__) || defined(BUILDING_WITH_CMAKE) > > #define WTF_USE_JSC 1 > #define WTF_USE_V8 0 This one looks wrong to me. Isn't there any port that might want WTF_USE_V8? Wasn't the EFL port recently adding support to use V8 instead of JSC? (In reply to comment #3) > (From update of attachment 97241 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=97241&action=review > > > Source/WebKit2/config.h:61 > > -#if defined(BUILDING_QT__) || defined(BUILDING_GTK__) > > +#if defined(BUILDING_QT__) || defined(BUILDING_GTK__) || defined(BUILDING_WITH_CMAKE) > > > > #define WTF_USE_JSC 1 > > #define WTF_USE_V8 0 > > This one looks wrong to me. Isn't there any port that might want WTF_USE_V8? Wasn't the EFL port recently adding support to use V8 instead of JSC? Right, it's not good, but I don't know that webkit2 supports v8 now. But, I'll check once more to find better way. BTW, WebKit/EFL doesn't support to use v8 yet. I am waiting Bug 61350 and need to clean up other local patches because it's too old. :( Thanks. Created attachment 97400 [details]
Patch
Created attachment 97402 [details]
Patch
Comment on attachment 97402 [details]
Patch
rs=me.
Comment on attachment 97402 [details] Patch Clearing flags on attachment: 97402 Committed r95005: <http://trac.webkit.org/changeset/95005> All reviewed patches have been landed. Closing bug. |