Bug 147450 - [EFL] Bump EFL version to 1.15.0
Summary: [EFL] Bump EFL version to 1.15.0
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Csaba Osztrogonác
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-30 11:39 PDT by Csaba Osztrogonác
Modified: 2015-08-06 04:11 PDT (History)
4 users (show)

See Also:


Attachments
Patch (1.99 KB, patch)
2015-07-30 11:41 PDT, Csaba Osztrogonác
no flags Details | Formatted Diff | Diff
Patch (1.99 KB, patch)
2015-08-06 00:12 PDT, Csaba Osztrogonác
no flags Details | Formatted Diff | Diff
Patch (3.04 KB, patch)
2015-08-06 01:46 PDT, Csaba Osztrogonác
gyuyoung.kim: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2015-07-30 11:39:22 PDT
EFL 1.15.0 is coming soon, after the release we should bump.

But unfortunately there is a build failure inside EFL 1.15.0-beta3:
bin/eeze/bin_eeze_eeze_scanner-eeze_scanner.o: In function `main':
/home/webkit/WebKit/WebKitBuild/DependenciesEFL/Source/efl-1.15.0-beta3/src/bin/eeze/eeze_scanner.c:427: undefined reference to `eeze_mount_tabs_watch'

It would be great if somebody could fix it before the release
or we should add the buildfix as a jhbuild module patch.
Comment 1 Csaba Osztrogonác 2015-07-30 11:41:06 PDT
Created attachment 257845 [details]
Patch

patch to show the build failure on the EWS bot with 1.15.0-beta3
Comment 2 Csaba Osztrogonác 2015-07-30 11:41:54 PDT
Gyuyoung, could you cc somebody responsible for EFL development?
Comment 3 Gyuyoung Kim 2015-07-30 18:08:59 PDT
CC'ing Ryuan and Juyoung Seo who work for EFL.
Comment 4 Gyuyoung Kim 2015-07-30 18:09:56 PDT
(In reply to comment #0)
> EFL 1.15.0 is coming soon, after the release we should bump.
> 
> But unfortunately there is a build failure inside EFL 1.15.0-beta3:
> bin/eeze/bin_eeze_eeze_scanner-eeze_scanner.o: In function `main':
> /home/webkit/WebKit/WebKitBuild/DependenciesEFL/Source/efl-1.15.0-beta3/src/
> bin/eeze/eeze_scanner.c:427: undefined reference to `eeze_mount_tabs_watch'
> 
> It would be great if somebody could fix it before the release
> or we should add the buildfix as a jhbuild module patch.

If EFL guys can't fix our issue, I agree with your suggestion to add a buildfix in jhbuild module.
Comment 5 Ryuan Choi 2015-07-31 08:27:42 PDT
(In reply to comment #4)
> (In reply to comment #0)
> > EFL 1.15.0 is coming soon, after the release we should bump.
> > 
> > But unfortunately there is a build failure inside EFL 1.15.0-beta3:
> > bin/eeze/bin_eeze_eeze_scanner-eeze_scanner.o: In function `main':
> > /home/webkit/WebKit/WebKitBuild/DependenciesEFL/Source/efl-1.15.0-beta3/src/
> > bin/eeze/eeze_scanner.c:427: undefined reference to `eeze_mount_tabs_watch'
> > 
> > It would be great if somebody could fix it before the release
> > or we should add the buildfix as a jhbuild module patch.
> 
> If EFL guys can't fix our issue, I agree with your suggestion to add a
> buildfix in jhbuild module.

I landed build fix on EFL trunk.
https://git.enlightenment.org/core/efl.git/commit/?id=bddce0b65047cdd6a9a32fbd05643bb27411f673

I believe that seoz will release 1.15 with it.
Comment 6 Csaba Osztrogonác 2015-08-06 00:12:33 PDT
Created attachment 258355 [details]
Patch

1.15 is out, let's try it
Comment 7 Gyuyoung Kim 2015-08-06 01:44:11 PDT
(In reply to comment #6)
> Created attachment 258355 [details]
> Patch
> 
> 1.15 is out, let's try it

Fail to include Eldbus.h.

../../Source/WebCore/platform/efl/BatteryProviderEfl.h:27:20: fatal error: Eldbus.h: No such file or directory
 #include <Eldbus.h>
Comment 8 Csaba Osztrogonác 2015-08-06 01:46:11 PDT
Created attachment 258360 [details]
Patch

with WebKit buildfix to be able include Eldbus.h
Comment 9 Gyuyoung Kim 2015-08-06 02:07:23 PDT
Comment on attachment 258360 [details]
Patch

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

> Source/WebKit2/PlatformEfl.cmake:601
> +

Unneeded lines.
Comment 10 Csaba Osztrogonác 2015-08-06 04:08:06 PDT
(In reply to comment #9)
> Comment on attachment 258360 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=258360&action=review
> 
> > Source/WebKit2/PlatformEfl.cmake:601
> > +
> 
> Unneeded lines.

Good point, will remove before landing.
Comment 11 Csaba Osztrogonác 2015-08-06 04:11:15 PDT
Committed r188038: <http://trac.webkit.org/changeset/188038>