Bug 104827 - [EFL] Build break with latest EFL libraries.
Summary: [EFL] Build break with latest EFL libraries.
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:
Blocks:
 
Reported: 2012-12-12 11:31 PST by Ryuan Choi
Modified: 2012-12-20 17:01 PST (History)
8 users (show)

See Also:


Attachments
first draft (26.49 KB, patch)
2012-12-12 12:41 PST, Ryuan Choi
no flags Details | Formatted Diff | Diff
Patch (27.10 KB, patch)
2012-12-20 07:33 PST, Ryuan Choi
no flags Details | Formatted Diff | Diff
patch for land (27.09 KB, patch)
2012-12-20 16:22 PST, 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 2012-12-12 11:31:15 PST
After EFL 1.8,
EFL libraries depends on Eo, which is newly added EFL library, and some of structures in Evas, Ecore are refactored.

So, in order to build with EFL trunk,
We should add Eo dependency and change declarations of Evas, Evas_Object, Ecore_Timer when detected EFL 1.8 .
Comment 1 Ryuan Choi 2012-12-12 12:41:33 PST
Created attachment 179104 [details]
first draft
Comment 2 Laszlo Gombos 2012-12-13 17:38:17 PST
I think 1.8 support can be added as experimental to WebKit trunk if building on 1.8 is stable and does not cause a large number of regressions (some number of regressions are probably expected). This is a decision however that constantly needs to be monitored and reevaluated as EFL dependencies evolve.

Ryuan, do you have any sort of test results running the EFL WebKit port using EFL 1.8/trunk ? 

Having said that I think the recommended stable version for WebKit development needs to stay 1.7.x for quite some time.
Comment 3 Ryuan Choi 2012-12-14 10:34:25 PST
(In reply to comment #2)
> I think 1.8 support can be added as experimental to WebKit trunk if building on 1.8 is stable and does not cause a large number of regressions (some number of regressions are probably expected). This is a decision however that constantly needs to be monitored and reevaluated as EFL dependencies evolve.
>
> Ryuan, do you have any sort of test results running the EFL WebKit port using EFL 1.8/trunk ? 
>
I tested both WebKit1 and WebKit2 layout test on 1.7 and latest trunk(which we can say initial code of EFL 1.8) and got a same results.

It's because several EFL libraries are just refactored to Eo based classes and our rendering is depends on cairo.
 
> Having said that I think the recommended stable version for WebKit development needs to stay 1.7.x for quite some time.

Absolutely right, now this is just for building webkit/efl on latest EFL trunk.

I am not sure, I heard that first snapshot of 1.8 are planned in next march.
This will be usefull for EFL developers who install latest EFL code and want not to use jhbuild.
Comment 4 Laszlo Gombos 2012-12-17 17:19:16 PST
Comment on attachment 179104 [details]
first draft

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

Make sense to me.

> Source/cmake/OptionsEfl.cmake:126
> +# Add Eo dependency if EFL version is equal to or greater 1.8
> +if (${EVAS_VERSION} VERSION_EQUAL 1.8 AND ${ECORE_VERSION} VERSION_EQUAL 1.8)

Comment does not seem to say what the code actually does. VERSION_EQUAL checks for "equal or greater" ?
Comment 5 Ryuan Choi 2012-12-20 07:33:00 PST
Created attachment 180339 [details]
Patch
Comment 6 Ryuan Choi 2012-12-20 16:22:47 PST
Created attachment 180436 [details]
patch for land
Comment 7 WebKit Review Bot 2012-12-20 17:00:56 PST
Comment on attachment 180436 [details]
patch for land

Clearing flags on attachment: 180436

Committed r138326: <http://trac.webkit.org/changeset/138326>
Comment 8 WebKit Review Bot 2012-12-20 17:01:01 PST
All reviewed patches have been landed.  Closing bug.