WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
160899
[EFL] Bump efl version from 1.17 to 1.18
https://bugs.webkit.org/show_bug.cgi?id=160899
Summary
[EFL] Bump efl version from 1.17 to 1.18
Gyuyoung Kim
Reported
2016-08-16 08:36:25 PDT
efl 1.18 is released recently. Let's use it for ewebkit.
Attachments
Patch
(4.08 KB, patch)
2016-08-16 08:38 PDT
,
Gyuyoung Kim
no flags
Details
Formatted Diff
Diff
Patch
(4.12 KB, patch)
2016-08-16 19:28 PDT
,
Gyuyoung Kim
no flags
Details
Formatted Diff
Diff
Patch
(4.16 KB, patch)
2016-08-16 19:38 PDT
,
Gyuyoung Kim
no flags
Details
Formatted Diff
Diff
Patch
(4.16 KB, patch)
2016-08-16 19:58 PDT
,
Gyuyoung Kim
no flags
Details
Formatted Diff
Diff
Patch
(5.22 KB, patch)
2016-08-16 21:31 PDT
,
Gyuyoung Kim
no flags
Details
Formatted Diff
Diff
Show Obsolete
(4)
View All
Add attachment
proposed patch, testcase, etc.
Gyuyoung Kim
Comment 1
2016-08-16 08:38:30 PDT
Created
attachment 286176
[details]
Patch
Gyuyoung Kim
Comment 2
2016-08-16 19:28:40 PDT
Created
attachment 286252
[details]
Patch
Gyuyoung Kim
Comment 3
2016-08-16 19:38:50 PDT
Created
attachment 286257
[details]
Patch
Gyuyoung Kim
Comment 4
2016-08-16 19:58:54 PDT
Created
attachment 286258
[details]
Patch
Gyuyoung Kim
Comment 5
2016-08-16 21:31:37 PDT
Created
attachment 286267
[details]
Patch
WebKit Commit Bot
Comment 6
2016-08-17 08:00:54 PDT
Comment on
attachment 286267
[details]
Patch Clearing flags on attachment: 286267 Committed
r204559
: <
http://trac.webkit.org/changeset/204559
>
WebKit Commit Bot
Comment 7
2016-08-17 08:00:58 PDT
All reviewed patches have been landed. Closing bug.
Csaba Osztrogonác
Comment 8
2016-08-17 10:23:25 PDT
It seems sudo Tools/efl/install-dependencies should be run on EFL bots after this patch.
Csaba Osztrogonác
Comment 9
2016-08-17 10:26:28 PDT
Comment on
attachment 286267
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=286267&action=review
> Tools/efl/jhbuild.modules:172 > - autogenargs="--disable-fribidi --disable-audio --disable-libmount --disable-physics --enable-lua-old --enable-i-really-know-what-i-am-doing-and-that-this-will-probably-break-things-and-i-will-fix-them-myself-and-send-patches-abb"> > + autogenargs="--disable-fribidi --disable-audio --disable-libmount --disable-physics --enable-i-really-know-what-i-am-doing-and-that-this-will-probably-break-things-and-i-will-fix-them-myself-and-send-patches-abb"> > <dependencies>
--enable-lua-old was added previously to be able to build EFL on AArch64, because LuaJIT wasn't supported on it that time. Do you know if it is supported now, or EFL intentionally dropped AArch64 build support?
> Tools/efl/jhbuild.modules:-180 > - <patch file="ecore_remove_warnings.patch" strip="1"/>
Please remove Tools/efl/patches/ecore_remove_warnings.patch file too if you don't need to patch EFL anymore.
Gyuyoung Kim
Comment 10
2016-08-17 18:06:32 PDT
(In reply to
comment #8
)
> It seems sudo Tools/efl/install-dependencies should be run > on EFL bots after this patch.
I missed to install the dependencies on perf bot. Now it is installed.
Gyuyoung Kim
Comment 11
2016-08-17 18:10:47 PDT
(In reply to
comment #9
)
> Comment on
attachment 286267
[details]
> Patch > > View in context: >
https://bugs.webkit.org/attachment.cgi?id=286267&action=review
> > > Tools/efl/jhbuild.modules:172 > > - autogenargs="--disable-fribidi --disable-audio --disable-libmount --disable-physics --enable-lua-old --enable-i-really-know-what-i-am-doing-and-that-this-will-probably-break-things-and-i-will-fix-them-myself-and-send-patches-abb"> > > + autogenargs="--disable-fribidi --disable-audio --disable-libmount --disable-physics --enable-i-really-know-what-i-am-doing-and-that-this-will-probably-break-things-and-i-will-fix-them-myself-and-send-patches-abb"> > > <dependencies> > > --enable-lua-old was added previously to be able to build EFL on AArch64, > because LuaJIT wasn't supported on it that time. Do you know if it is > supported now, or EFL intentionally dropped AArch64 build support?
I was noticed it when bumping efl. There is build break when using "--enable-lua-old" option with efl-1.18. That's why I removed it. Anyway I don't have to build EFL on ARM environment. So could you check it if current version can be built on ARM ?
> > Tools/efl/jhbuild.modules:-180 > > - <patch file="ecore_remove_warnings.patch" strip="1"/> > > Please remove Tools/efl/patches/ecore_remove_warnings.patch file too if you > don't need to patch EFL anymore.
ok, let me remove it or update it.
Csaba Osztrogonác
Comment 12
2016-08-22 02:53:59 PDT
(In reply to
comment #11
)
> (In reply to
comment #9
) > > Comment on
attachment 286267
[details]
> > Patch > > > > View in context: > >
https://bugs.webkit.org/attachment.cgi?id=286267&action=review
> > > > > Tools/efl/jhbuild.modules:172 > > > - autogenargs="--disable-fribidi --disable-audio --disable-libmount --disable-physics --enable-lua-old --enable-i-really-know-what-i-am-doing-and-that-this-will-probably-break-things-and-i-will-fix-them-myself-and-send-patches-abb"> > > > + autogenargs="--disable-fribidi --disable-audio --disable-libmount --disable-physics --enable-i-really-know-what-i-am-doing-and-that-this-will-probably-break-things-and-i-will-fix-them-myself-and-send-patches-abb"> > > > <dependencies> > > > > --enable-lua-old was added previously to be able to build EFL on AArch64, > > because LuaJIT wasn't supported on it that time. Do you know if it is > > supported now, or EFL intentionally dropped AArch64 build support? > > I was noticed it when bumping efl. There is build break when using > "--enable-lua-old" option with efl-1.18. That's why I removed it. Anyway I > don't have to build EFL on ARM environment. So could you check it if current > version can be built on ARM ?
I have only old AArch64 systems (14.04/14.10) which don't have LuaJIT at all, so the current version won't build as is. I tried with --enable-lua-old, but I think I got the same warning you got: lib/edje/edje_private.h:33:17: fatal error: lua.h: No such file or directory #include <lua.h> It is obviously a bug in the EFL configure scripts.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug