Bug 83579

Summary: [CMake] Use WEBKIT_CREATE_FORWARDING_HEADERS to generate WK2 forwarding headers
Product: WebKit Reporter: Patrick R. Gansterer <paroga>
Component: New BugsAssignee: Patrick R. Gansterer <paroga>
Status: REOPENED ---    
Severity: Normal CC: dbates, gyuyoung.kim, rakuco, ryuan.choi, tmpsantos, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 83574, 92630, 92663    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch none

Description Patrick R. Gansterer 2012-04-10 07:17:39 PDT
[CMake] Use WEBKIT_CREATE_FORWARDING_HEADERS to generate WK2 forwarding headers
Comment 1 Patrick R. Gansterer 2012-04-10 07:21:55 PDT
Created attachment 136453 [details]
Patch
Comment 2 Raphael Kubo da Costa (:rakuco) 2012-04-10 09:05:47 PDT
CC'ing the Samsung guys who can test it with their working WebKit2-EFL.
Comment 3 Raphael Kubo da Costa (:rakuco) 2012-04-24 16:11:11 PDT
Given the lack of feedback and the fact that webkit2-efl hasn't been fully upstreamed yet (IIRC), I think you can just go forward with it.
Comment 4 Ryuan Choi 2012-04-24 17:09:22 PDT
Sorry about late answer.
I did not catch this bug.

As kubo commented, more patches are needed to build webkit2-efl successfully.
So, I do not have objection for this.

But FYI, I have some build break of webkit2-efl in my local(git@github.com:bunhere/webkit-mirror.git webkit-efl-new branch).
Below is error message.

/home/ryuan/workspace/github/webkit-mirror/Source/WebKit2/Platform/CoreIPC/Connection.cpp:31:29: fatal error: WebCore/RunLoop.h: No such file or directory
compilation terminated.
[ 89%] [ 89%] Building CXX object Source/WebKit2/CMakeFiles/ewebkit2.dir/Shared/APIObject.cpp.o
Building CXX object Source/WebKit2/CMakeFiles/ewebkit2.dir/Shared/APIClientTraits.cpp.o
[ 89%] Building CXX object Source/WebKit2/CMakeFiles/ewebkit2.dir/Shared/ChildProcess.cpp.o
In file included from /home/ryuan/workspace/github/webkit-mirror/build/DerivedSources/ForwardingHeaders/WebKit2/WKBase.h:1:0,
                 from /home/ryuan/workspace/github/webkit-mirror/Source/WebKit2/WebProcess/InjectedBundle/API/c/WKBundlePage.h:29,
                 from /home/ryuan/workspace/github/webkit-mirror/Source/WebKit2/Shared/APIClientTraits.h:29,
                 from /home/ryuan/workspace/github/webkit-mirror/Source/WebKit2/Shared/APIClientTraits.cpp:27:
/home/ryuan/workspace/github/webkit-mirror/Source/WebKit2/Shared/API/c/WKBase.h:41:31: fatal error: WebKit2/WKBaseEfl.h: No such file or directory
compilation terminated.
make[2]: *** [Source/WebKit2/CMakeFiles/ewebkit2.dir/Shared/APIClientTraits.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
In file included from /home/ryuan/workspace/github/webkit-mirror/Source/WebKit2/Shared/ChildProcess.cpp:27:0:
/home/ryuan/workspace/github/webkit-mirror/Source/WebKit2/Shared/ChildProcess.h:30:29: fatal error: WebCore/RunLoop.h: No such file or directory
compilation terminated.
make[2]: *** [Source/WebKit2/CMakeFiles/ewebkit2.dir/Platform/CoreIPC/Connection.cpp.o] Error 1
make[2]: *** [Source/WebKit2/CMakeFiles/ewebkit2.dir/Shared/ChildProcess.cpp.o] Error 1
make[1]: *** [Source/WebKit2/CMakeFiles/ewebkit2.dir/all] Error 2
make: *** [all] Error 2
Comment 5 Ryuan Choi 2012-04-24 18:36:25 PDT
I checked and fixed build break via http://pastebin.com/V5v3tA8R
Comment 6 Patrick R. Gansterer 2012-07-29 14:23:00 PDT
Created attachment 155183 [details]
Patch
Comment 7 Patrick R. Gansterer 2012-07-29 14:47:39 PDT
Comment on attachment 155183 [details]
Patch

Clearing flags on attachment: 155183

Committed r123983: <http://trac.webkit.org/changeset/123983>
Comment 8 Patrick R. Gansterer 2012-07-29 14:47:50 PDT
All reviewed patches have been landed.  Closing bug.
Comment 9 Thiago Marcos P. Santos 2012-07-30 01:52:05 PDT
This patch broke the EFL build. It wasn't caught by the EWS because it would need a clean build to reproduce the breakage.
Comment 10 Patrick R. Gansterer 2012-07-30 01:56:05 PDT
(In reply to comment #9)
> This patch broke the EFL build. It wasn't caught by the EWS because it would need a clean build to reproduce the breakage.

Would be great if you can post the error messages too, if you want someone to fix it. ;-)
Comment 11 Thiago Marcos P. Santos 2012-07-30 05:15:54 PDT
(In reply to comment #10)
> (In reply to comment #9)
> > This patch broke the EFL build. It wasn't caught by the EWS because it would need a clean build to reproduce the breakage.
> 
> Would be great if you can post the error messages too, if you want someone to fix it. ;-)

I'm fixing it.
Comment 12 WebKit Review Bot 2012-07-30 11:31:28 PDT
Re-opened since this is blocked by 92663