Bug 135636 - Build break since r172093
Summary: Build break since r172093
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Ryuan Choi
URL:
Keywords:
Depends on: 135620
Blocks:
  Show dependency treegraph
 
Reported: 2014-08-05 19:27 PDT by Ryuan Choi
Modified: 2014-08-06 02:57 PDT (History)
7 users (show)

See Also:


Attachments
first trial (7.11 KB, patch)
2014-08-05 19:37 PDT, Ryuan Choi
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryuan Choi 2014-08-05 19:27:56 PDT
r172093 Added AbstractView.idl and CodeGeneratorJS.pm does not take care of it.

In file included from /home/ryuan/workspace/project/webkit/efl-webkit/WebKitBuild/Release/DerivedSources/WebCore/JSAbstractView.cpp:22:0:
/home/ryuan/workspace/project/webkit/efl-webkit/WebKitBuild/Release/DerivedSources/WebCore/JSAbstractView.h:24:26: fatal error: AbstractView.h: No such file or directory
 #include "AbstractView.h"
                          ^
compilation terminated.
Comment 1 Ryuan Choi 2014-08-05 19:37:13 PDT
Created attachment 236075 [details]
first trial
Comment 2 Laszlo Gombos 2014-08-05 20:15:12 PDT
Do we know why the try bot did not catch this ? It was green for efl-wk2 at https://bugs.webkit.org/show_bug.cgi?id=135620
Comment 3 Gyuyoung Kim 2014-08-05 20:36:37 PDT
(In reply to comment #2)
> Do we know why the try bot did not catch this ? It was green for efl-wk2 at https://bugs.webkit.org/show_bug.cgi?id=135620

It looks efl-ews didn't generate new file because of missing to remove cache files, which was used for reducing build time. I just fixed it.
Comment 4 Gyuyoung Kim 2014-08-05 21:42:11 PDT
Comment on attachment 236075 [details]
first trial

View in context: https://bugs.webkit.org/attachment.cgi?id=236075&action=review

rs=me to fix build break on EFL port.

> Source/WebCore/ChangeLog:3
> +        [EFL] Build break since r172093

How about using "Build break on EFL port since r172093". Because this patch modifies code generation script, and it is not EFL port specific. [EFL] prefix may cause that this patch looks like EFL specific patch.
Comment 5 Ryuan Choi 2014-08-05 22:21:06 PDT
Committed r172128: <http://trac.webkit.org/changeset/172128>
Comment 6 Ryuan Choi 2014-08-05 22:22:47 PDT
Comment on attachment 236075 [details]
first trial

Clearing flags after landed manually.

I am not 100% sure that this is the best.
I landed this because all bots are green and it fixes real build break.
Comment 7 Csaba Osztrogonác 2014-08-06 02:57:06 PDT
(In reply to comment #5)
> Committed r172128: <http://trac.webkit.org/changeset/172128>

It broke the bindings-generation tests:
http://build.webkit.org/builders/Apple%20Mavericks%20Release%20WK1%20%28Tests%29/builds/7827