This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

• patch (patch_bug_4041_reload_egl_libs.diff, text/plain, 2019-05-24 15:09:48 +0000, 539 bytes)

Reported in version: don't know
Reported for operating system, platform: Android (All), All

Comments on the original bug report:

On 2018-01-11 09:57:41 +0000, Sylvain wrote:

On Android, if you set no attribute using SDL_GL_SetAttribute(), and try to create a SDL Render OpenGLES 1:

• it loads first by default GLESv2 libraries
• creates the rendere OpenGLES 1
• recreates the Window to have a context 1.1 ( https://hg.libsdl.org/SDL/file/00fb5966c44f/src/render/opengles/SDL_render_gles.c#l298 )

But it doesn't unload libraries, then reload GLESv1 lib. So the SDL_Renderer OpenGLES 1 is working with GLES 2 libs, which seems inconsistent.

If you, at first, set
SDL_GL_SetAttribute(SDL_GL_CONTEXT_PROFILE_MASK, SDL_GL_CONTEXT_PROFILE_ES);
SDL_GL_SetAttribute(SDL_GL_CONTEXT_MAJOR_VERSION, 1);
SDL_GL_SetAttribute(SDL_GL_CONTEXT_MINOR_VERSION, 1);
It will correctly load GLES v1 libraries.

On 2018-01-11 10:02:42 +0000, Sylvain wrote:

See https://discourse.libsdl.org/t/android-update-breaks-opengl-es-1-0/23709

On 2019-05-18 18:48:53 +0000, Ryan C. Gordon wrote:

Tagging a bunch of bugs with "target-2.0.10" so we have a clear list of things to address before a 2.0.10 release.

Please note that "addressing" one of these bugs might mean deciding to defer on it until after 2.0.10, or resolving it as WONTFIX, etc. This is just here to tell us we should look at it carefully, and soon.

If you have new information or feedback on this issue, this is a good time to add it to the conversation, as we're likely to be paying attention to this specific report in the next few days/weeks.

Thanks!

--ryan.

On 2019-05-24 15:09:48 +0000, Sylvain wrote:

Created attachment 3791
patch

Here's a small patch to reload egl libs when SDL_RecreateWindow() is called.
It fixes the issue, also the case from bug 4042

( SDL_RecreateWindow() is used by SDL_Renderer gl, gles1, gles2. )

But maybe there is some case where we want to keep the previous libs ?

On 2019-06-09 01:40:32 +0000, Sam Lantinga wrote:

Patch added, thanks!
https://hg.libsdl.org/SDL/rev/00a1aa5a1fc0