Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow for X11-less egl builds on linux #80

Closed
wants to merge 3 commits into from
Closed

Allow for X11-less egl builds on linux #80

wants to merge 3 commits into from

Conversation

gustavoz
Copy link

This patch series allows a cross-compiled EGL X11-less build for linux.
It makes sense for embedded targets where you don't want to pull in a big chunk of X11 stack to build gtk3 with wayland for example (via mesa EGL).

We might be cross-compiling and hence host might be != target.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
Since libgtk3 uses libepoxy and can be built for wayland alone without
X11 support it doesn't make sense to force OpenGL support and a full
X11 stack.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
When building mesa egl without x11 and gles2 the headers need a
MESA_EGL_NO_X11_HEADERS define, so put them in epoxy.pc as well since
otherwise builds will fail.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
@jeremyhu
Copy link

I think pull request #52 handles this a bit cleaner. Is there something about that change that doesn't work well for you.

@gustavoz
Copy link
Author

gustavoz commented Feb 5, 2016

You still need gustavoz@68e5f15 since otherwise using non-glx mesa with libepoxy will fail to build properly (or rather libgtk3 with libepoxy in that scenario), like so:

.../sysroot/usr/include/EGL/eglplatform.h:119:22: fatal error: X11/Xlib.h: No such file or directory

Another matter would be to build egl-less glx libepoxy, but that wasn't addressed by me either.

@sardemff7
Copy link

Any news on these?

@gustavoz
Copy link
Author

I've already landed all the required patches in buildroot to make this possible, take a look at https://git.busybox.net/buildroot/tree/package/libepoxy
If this hits vanilla libepoxy some day i'll be happy to update/remove the patches.

@@ -102,6 +102,8 @@ if test x$build_egl = xyes; then
AC_DEFINE([BUILD_EGL], [1], [build EGL tests])
fi

AC_SUBST(EGL_CFLAGS)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need for that, PKG_CHECK_MODULES does it already.

@ebassi ebassi closed this in ec93dcd Feb 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants