Bug 113403 - [EFL][jhbuild] Evas fails to build when Wayland is present on the system
Summary: [EFL][jhbuild] Evas fails to build when Wayland is present on the system
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Raphael Kubo da Costa (:rakuco)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-27 06:42 PDT by Raphael Kubo da Costa (:rakuco)
Modified: 2013-03-27 08:35 PDT (History)
1 user (show)

See Also:


Attachments
Patch (1.94 KB, patch)
2013-03-27 06:45 PDT, Raphael Kubo da Costa (:rakuco)
no flags Details | Formatted Diff | Diff
Patch (2.02 KB, patch)
2013-03-27 08:19 PDT, Raphael Kubo da Costa (:rakuco)
kenneth: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Raphael Kubo da Costa (:rakuco) 2013-03-27 06:42:19 PDT
As of Evas 1.7.5, when Wayland is detected on the system, support for both Wayland SHM and Wayland EGL is automatically enabled in autoconf.

The code, however, seems to depend on a few other flags which must be explicitly enabled (namely, --enable-gl-flavor-gles, --enable-gles-variety-sgx and --enable-gl-xcb). Otherwise, we get this error:

  evas_wl_main.c: In function 'eng_window_new':
  evas_wl_main.c:197:18: error: 'Evas_Engine_GL_Context' has no member named 'egldisp'

The flags we need to pass are the ones mentioned in bug 112522, which we closed as WONTFIX. The problem here is different though, and according to one of the Wayland-EFL developers enabling those flags even if the system does not have Wayland or GL support should be fine.
Comment 1 Raphael Kubo da Costa (:rakuco) 2013-03-27 06:43:11 PDT
(In reply to comment #0)
> The problem here is different though, and according to one of the Wayland-EFL developers enabling those flags even if the system does not have Wayland or GL support should be fine.

Kalyan, can you double-check that what I wrote above makes sense?
Comment 2 Raphael Kubo da Costa (:rakuco) 2013-03-27 06:45:06 PDT
Created attachment 195304 [details]
Patch
Comment 3 Kalyan 2013-03-27 06:52:45 PDT
Comment on attachment 195304 [details]
Patch

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

> Tools/efl/jhbuild.modules:197
> +             autogenargs="--enable-gl-xcb --enable-gles-variety-sgx --enable-gl-flavor-gles">

This would be an issue if the underlying system doesn't support EGL i.e Nvidia drivers. Have you checked this ??

I also remember that I couldn't get evas build (when libwayland-dev in my system) as it checked for some non existent packages. I had to remove wayland related stuff before I could get it compiled. I can cross check this one again though.
Comment 4 Kalyan 2013-03-27 06:54:47 PDT
Comment on attachment 195304 [details]
Patch

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

>> Tools/efl/jhbuild.modules:197
>> +             autogenargs="--enable-gl-xcb --enable-gles-variety-sgx --enable-gl-flavor-gles">
> 
> This would be an issue if the underlying system doesn't support EGL i.e Nvidia drivers. Have you checked this ??
> 
> I also remember that I couldn't get evas build (when libwayland-dev in my system) as it checked for some non existent packages. I had to remove wayland related stuff before I could get it compiled. I can cross check this one again though.

k, first issue might be obsolete as per your earlier comments.
Comment 5 Raphael Kubo da Costa (:rakuco) 2013-03-27 08:19:53 PDT
Created attachment 195329 [details]
Patch
Comment 6 Raphael Kubo da Costa (:rakuco) 2013-03-27 08:21:02 PDT
(In reply to comment #5)
> Created an attachment (id=195329) [details]
> Patch

Reverse approach now: building with those additional flags made MiniBrowser simply crash or hang when being launched.

It's safer to just disable Wayland while we don't investigate and/or add proper support for it.
Comment 7 Kalyan 2013-03-27 08:24:44 PDT
LGTM
Comment 8 Raphael Kubo da Costa (:rakuco) 2013-03-27 08:35:47 PDT
Committed r146988: <http://trac.webkit.org/changeset/146988>