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

sdl consumer may require SDL12COMPAT_NO_QUIT_VIDEO #971

Open
schauveau opened this issue Jan 21, 2024 · 2 comments
Open

sdl consumer may require SDL12COMPAT_NO_QUIT_VIDEO #971

schauveau opened this issue Jan 21, 2024 · 2 comments

Comments

@schauveau
Copy link

SDL1 is often being replaced by https://github.com/libsdl-org/sdl12-compat , a compatibility wrapper around SDL2.

Applications using the sdl consumer may experience some problems as shown in this Flowblade bug report jliljebl/flowblade#1134

The solution appears to be SDL12COMPAT_NO_QUIT_VIDEO=1, a currently undocumented environment variable of sdl12-compat. libsdl-org/sdl12-compat#329

@ddennedy
Copy link
Member

The sdl consumer is going to be deprecated. That does not mean immediate removal, but the writing is on the wall... that some day.

@jliljebl It does not like it would be very difficult for you to make and test a simple change to move SDL_Quit() from consumer_stop() to consumer_close(). Alternatively, it looks like you can set the consumer property sdl_started=1 before calling stop() on it from python to avoid the error. I am not in a position to do this. Someone who cares about sdl should.

@jliljebl
Copy link
Contributor

Ok, I'll work on this in October and see how this goes. I'm moving to a new place in a few weeks and that will probably eat up a lot of my time until maybe second week of October.

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

No branches or pull requests

3 participants