Fix SDL threading on macOS/Linux.#107963
Conversation
| "thread/SDL_thread.c", | ||
| "thread/generic/SDL_syscond.c", | ||
| "thread/generic/SDL_sysrwlock.c", | ||
| "thread/generic/SDL_systhread.c", |
There was a problem hiding this comment.
This was likely the main issue, it seems like I saw it included in the Windows driver:
thirdparty/sdl/thread/windows/SDL_systhread.c
29:#include "SDL_systhread_c.h"
But I failed to realize this is thread/windows/SDL_systhread_c.h and not thread/generic/SDL_systhread.c.
It's super confusing that they have files with the same names in different folders (so they seem mutually exclusive) but then the Windows one depends on a couple thread/generic files anyway...
|
I also fixed the SDL update script to take this into account and document it better, and add the pkgconfig Linux fixup for #106218 (comment). |
|
lol what a mess: So we shouldn't build And same issue for the Windows part: So I guess I'll add back the header but without the corresponding C file to avoid the risk of compiling it... this could definitely be improved somewhat upstream to be more self-contained. |
And fixup pkg-config check for SDL on Linux. Co-authored-by: Rémi Verschelde <rverschelde@gmail.com>
akien-mga
left a comment
There was a problem hiding this comment.
Seems to work fine on Linux, tested with an Xbox Series controller on bluetooth.
|
Thanks! |
These files are used only on Windows and seems to cause infinite loop on macOS.
Bugsquad edit: