Skip to content

More robust EGL display initilization. #550

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 2 commits into
base: master
Choose a base branch
from

Conversation

tritao
Copy link
Contributor

@tritao tritao commented Feb 13, 2025

@VolkerEnderlein Before merging lets wait until folks confirm it fixes the issue.

Based on a patch by @lepalom.

@tritao
Copy link
Contributor Author

tritao commented Feb 13, 2025

@Rexbas @lepalom Can you try this?

@Rexbas
Copy link
Contributor

Rexbas commented Feb 13, 2025

I still get this:
Coin warning in cc_glglue_instance(): Error when setting up the GL context. This can happen if there is no current context, or if the context has been set up incorrectly.

But now I also get this message. I did not get this previously with the NVIDIA proprietary drivers:
libEGL warning: egl: failed to create dri2 screen

I remember that without the NVIDIA proprietary drivers, I also got this message. So I think the PR might be some positive progress.

Also now I get a black FreeCAD instead of white:
image

Aside from the dri2 screen message, Hello Cone is still the same (no cone).

@tritao
Copy link
Contributor Author

tritao commented Feb 13, 2025

I still get this: Coin warning in cc_glglue_instance(): Error when setting up the GL context. This can happen if there is no current context, or if the context has been set up incorrectly.

But now I also get this message. I did not get this previously with the NVIDIA proprietary drivers: libEGL warning: egl: failed to create dri2 screen

I remember that without the NVIDIA proprietary drivers, I also got this message. So I think the PR might be some positive progress.

Also now I get a black FreeCAD instead of white: image

Aside from the dri2 screen message, Hello Cone is still the same (no cone).

I think our best bet is to duplicate whatever logic SDL or GLFW do to set up EGL: https://github.com/glfw/glfw/blob/master/src/egl_context.c

At the same time it makes me think I'd rather have Coin not need to deal directly with such logic and leave it to windowing libraries which already do a perfect fine job.

@lepalom
Copy link

lepalom commented Feb 15, 2025

@tritao I have tested your fork, the egl-fixes and I see a black cone :-( it is not working properly ... I really don't know what to do.

@lepalom
Copy link

lepalom commented Mar 9, 2025

Anton Gladky solved, at leat for Debian, could you check it?
https://salsa.debian.org/science-team/coin3/-/blob/master/debian/patches/Egl_dust_off2.patch?ref_type=heads
@tritao

@heiserhorn
Copy link

@lepalom I am running Debian sid with the latest version of coin3 with the patch mentioned above but I have exactly the same problem. I have commented on the debian closed bug which should be reopened.

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.

4 participants