Skip to content

Allow to build PrusaSlicer with EGL support #14214

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

cryptomilk
Copy link
Contributor

A lot of Linux distribution don't build wxWidgets with EGL support yet. This is why this option defaults to OFF. If a Linux distribution has built wxWidgets and GLEW with EGL support, you should turn that on!

openSUSE and Fedora are working on building wxWidgets with EGL support:

https://bugzilla.redhat.com/show_bug.cgi?id=2336093
https://bugzilla.suse.com/show_bug.cgi?id=1235150

The patch allows you to enable EGL support with: cmake -DSLIC3R_EGL=ON

@cryptomilk
Copy link
Contributor Author

Rebased

@cryptomilk
Copy link
Contributor Author

FYI: PrusaSlicer on openSUSE Tumbleweed is built with EGL/Wayland support now.

Multi-Language Vulkan/GL/GLES/EGL/GLX/WGL Loader-Generator based on the
official specs.

https://github.com/Dav1dde/glad

Signed-off-by: Andreas Schneider <[email protected]>
As static linking is used everywhere we can link GLAD only once.

Signed-off-by: Andreas Schneider <[email protected]>
As libvgcode already uses GLAD there is no need for GLEW, which is
unmaintained anyway.

Signed-off-by: Andreas Schneider <[email protected]>
As libvgcode already uses GLAD there is no need for GLEW, which is
unmaintained anyway.

Signed-off-by: Andreas Schneider <[email protected]>
Signed-off-by: Andreas Schneider <[email protected]>
A lot of Linux distribution don't build wxWidgets with EGL support yet.
This is why by default this option is set to OFF. If a Linux
distribution has built wxWidgets and GLEW with EGL support, you should
turn that on!
@cryptomilk
Copy link
Contributor Author

This requires #14440 to go in first.

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.

1 participant