ANGLE's include/EGL/eglplatform.h does require X11 headers. I managed to work around this by replacing it by MESA's include/EGL/eglplatform.h (surprisingly, since it requires MESA_EGL_NO_X11_HEADERS defined in order not to require them).
I am also facing this issue when building WebKitGTK+ 2.16.5 using
Buildroot with Wayland enabled and X.org disabled. Better than
replacing the “eglplatform.h” header with the Mesa version, I can
try to make a patch which includes the X11 headers when ANGLE_USE_X11
is defined, and otherwise provides the same definitions as the Mesa
version when MESA_EGL_NO_X11_HEADERS is defined. Probably it is also
a good idea to add the definitions for Wayland when WL_EGL_PLATFORM
Created attachment 315227 [details]
Note that there are important steps to take when updating ANGLE. See http://trac.webkit.org/wiki/UpdatingANGLE
Comment on attachment 315227 [details]
Attachment 315227 [details] did not pass mac-ews (mac):
New failing tests:
Created attachment 315234 [details]
Archive of layout-test-results from ews100 for mac-elcapitan
The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100 Port: mac-elcapitan Platform: Mac OS X 10.11.6
You'll need to update the ANGLE patch (changes.diff)
Created attachment 315241 [details]
(In reply to Adrian Perez from comment #7)
> Created attachment 315241 [details]
This new version of the patch also updates “changes.diff” — thanks for the
I am doing a Wayland-only build right now in Buildroot to double-check that
the patch works as intended. Let's wait until it finishes before landing,
and wait for reviewer's comments in the meanwhile.
(In reply to Adrian Perez from comment #8)
> I am doing a Wayland-only build right now in Buildroot to double-check that
> the patch works as intended. Let's wait until it finishes before landing,
> and wait for reviewer's comments in the meanwhile.
The build finished fine with this patch applied. Additionally I needed to
pass the following to CMake in the Buildroot recipe to avoid the system
Mesa headers including X11 headers:
The above is needed because this patch affects only the copy included within
ANGLE but not the system (in this case, sysroot-wide) EGL header. I think this
is fine and I'll wait until tomorrow to land this patch, just in case anybody
wants to comment, and also that way I can keep an eye on the buildbots (right
now is very late for me).
Comment on attachment 315241 [details]
Clearing flags on attachment: 315241
Committed r219446: <http://trac.webkit.org/changeset/219446>
All reviewed patches have been landed. Closing bug.
The patch was also attached to a bug report in the upstream bug tracker,
as recommended by the guidelines for “Source/ThirdParty/ANGLE/”:
(In reply to Adrian Perez from comment #12)
> The patch was also attached to a bug report in the upstream bug tracker,
> as recommended by the guidelines for “Source/ThirdParty/ANGLE/”:
The patch (with small modifications) was merged upstream in ANGLE, yay!
On revision r225340 <https://trac.webkit.org/r225340> ANGLE was updated.
The Wayland EGL definitions are on the new eglplatform.h file, but not the check for MESA_EGL_NO_X11_HEADERS
I guess that is enough, right?
Is RESOLVED FIXED correct? I don't know which ANGLE revision is in the current webkit-gtk tarballs, but I can say for sure that the eglplatform.h that comes with current webkit is still "broken" and needs to be replaced by the MESA one form me.
(In reply to ManDay from comment #15)
> Is RESOLVED FIXED correct? I don't know which ANGLE revision is in the
> current webkit-gtk tarballs, but I can say for sure that the eglplatform.h
> that comes with current webkit is still "broken" and needs to be replaced by
> the MESA one form me.
Yes, the change that went in the patch that the ANGLE maintainers merged
upstream is lacking one part that I had made originally to fix this bug.
I have opened bug #189844 to re-add the needed part in our bundled copy
of ANGLE because now I am hitting this issue again.