RESOLVED FIXED 62692
[CMAKE][WK2] include cmakeconfig.h in WebKit2/config.h 
https://bugs.webkit.org/show_bug.cgi?id=62692
Summary [CMAKE][WK2] include cmakeconfig.h in WebKit2/config.h
Ryuan Choi
Reported Wednesday, June 15, 2011 4:08:20 AM UTC
Patch will be updated.
Attachments
Patch (1.32 KB, patch)
2011-06-14 23:50 PDT, Ryuan Choi 		no flags
DetailsFormatted DiffDiff
Patch (2.09 KB, patch)
2011-06-15 22:27 PDT, Ryuan Choi 		no flags
DetailsFormatted DiffDiff
Patch (2.03 KB, patch)
2011-06-15 22:37 PDT, Ryuan Choi 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Ryuan Choi
Comment 1 Wednesday, June 15, 2011 7:50:05 AM UTC
Created attachment 97241 [details] Patch
Raphael Kubo da Costa (:rakuco)
Comment 2 Wednesday, June 15, 2011 1:57:59 PM UTC
Looks OK to me, informal r+.
Lucas De Marchi
Comment 3 Wednesday, June 15, 2011 3:08:03 PM UTC
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?
Ryuan Choi
Comment 4 Thursday, June 16, 2011 12:36:15 AM UTC
(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.
Ryuan Choi
Comment 5 Thursday, June 16, 2011 6:27:56 AM UTC
Created attachment 97400 [details] Patch
Ryuan Choi
Comment 6 Thursday, June 16, 2011 6:37:00 AM UTC
Created attachment 97402 [details] Patch
Eric Seidel (no email)
Comment 7 Monday, September 12, 2011 11:36:32 PM UTC
Comment on attachment 97402 [details] Patch rs=me.
WebKit Review Bot
Comment 8 Tuesday, September 13, 2011 4:29:50 AM UTC
Comment on attachment 97402 [details] Patch Clearing flags on attachment: 97402 Committed r95005: <http://trac.webkit.org/changeset/95005>
WebKit Review Bot
Comment 9 Tuesday, September 13, 2011 4:29:56 AM UTC
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.