-
Notifications
You must be signed in to change notification settings - Fork 109
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
Missing EGL support for Wayland #451
Missing EGL support for Wayland #451
Comments
Hello, |
My knowledge about GLX and Wayland is very basic (only from a user perspective), so I do not know what issues may arise when applying this patch. I could make the removed check for GLX optional and enabled by default. Maybe some other devs can shed some light on this. |
I've found why coin3d doesn't work correctly on the Wayland environment. The QGLWidget has a problem with a Wayland. I've re-implemented the QuarterWidget with QOpenGLWidget ( https://github.com/epuzanov/pivy/tree/QOpenGLWidget ). |
This bug is going to get really noisy soon due FreeCAD wayland users preferring to operate in wayland env and not Xwindows |
I updated an earlier branch to fix some issues I encountered while investigating EGL support on Wayland. You can find this work at https://github.com/magic-lantern-studio/coin/tree/glue_egl. I did not create a merge request since it has not been tested for building on other platforms, other than Ubuntu. Information on building this can be found at https://github.com/magic-lantern-studio/coin/wiki/OpenGL-ES-Investigation. |
It's happening |
It really is happening |
@VolkerEnderlein are there plans to review #471 ? |
My two cents: #471 is still in draft and marked "do not merge", and now needs a rebase. I think the best would be to get that PR dusted off, apply b49b8b4, pull it into a separate branch and then hopefully CI will build testable binaries. If it doesn't, the PR should be extended so that it does. These should then be tested by the parties that need this (FreeCAD), then they can determine whether it is ready to be merged or not. |
see this thread in coin3d-discuss
The text was updated successfully, but these errors were encountered: