mltframework/mlt

sdl consumer may require SDL12COMPAT_NO_QUIT_VIDEO

schauveau opened this issue · 3 comments

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

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.

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.

Now that we have a fix for embedding sdl2 consumer 7acb1f1
this will not be address in sdl (1.2) consumer as it has been deprecated ind540f8f73a73087fde2126db0618d4622c6b837a