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

Implement an EGL driver #68944

Closed
wants to merge 1 commit into from
Closed

Conversation

Riteo
Copy link
Contributor

@Riteo Riteo commented Nov 21, 2022

This PR isn't meant to be merged as-is, as no current code references it.

This is needed by Wayland and could potentially be used by other platforms too, such as Android.

This driver has been written in a platform-agnostic way, requiring the implementing platform to create a platform-specific manager inheriting from the EGLManager class, implementing the _get_platform_extension_* methods.

It also includes its GLAD loader, currently only checking the Wayland platform extension.

@Riteo
Copy link
Contributor Author

Riteo commented Nov 21, 2022

Marking as a draft until I'm sure that EGL loading works well enough.

@Riteo Riteo marked this pull request as draft November 21, 2022 12:33
@Riteo
Copy link
Contributor Author

Riteo commented Nov 21, 2022

Seems to work fine now after a brief testing.

@Riteo Riteo marked this pull request as ready for review November 21, 2022 19:17
@Riteo
Copy link
Contributor Author

Riteo commented Nov 21, 2022

Seems to work fine now after a brief testing.

@Riteo Riteo force-pushed the egl-manager-only branch 2 times, most recently from 905902e to 84d14a8 Compare December 6, 2022 22:52
This is needed by Wayland and could potentially be used by other
platforms too, such as Android.

This driver has been written in a platform-agnostic way, requiring the
implementing platform to create a platform-specific manager inheriting
from the EGLManager class, implementing the `_get_platform_extension_*`
methods.

It also includes its own GLAD loader and some extra logic for loading it
automatically.
@stalkerg
Copy link

Is it will be merged when #57025 will be ready?

@Riteo
Copy link
Contributor Author

Riteo commented Sep 17, 2023

@stalkerg well it's needed for the Wayland backend to work so in one way or another... yes.

Note though that this code has been also merged into #72831 which will probably get merged way before the Wayland PR is ready (time is little and this stuff takes time to polish), so this one might be closed if that happens.

@Anutrix
Copy link
Contributor

Anutrix commented Sep 22, 2023

Since #72831 got merged, maybe we can rebase and see if there's any diff.

@Riteo
Copy link
Contributor Author

Riteo commented Sep 22, 2023

@Anutrix that PR is based on this one with a few additions, so there shouldn't be any considerable differences (outside of the aforementioned additions).

This PR is effectively superseded by #72831, which basically includes this code, so I'm closing this one.

@Riteo Riteo closed this Sep 22, 2023
@AThousandShips AThousandShips removed this from the 4.2 milestone Sep 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants