Skip to content

sdl2: Add patch fixing non-threaded emscripten build#26070

Merged
ras0219-msft merged 1 commit into
microsoft:masterfrom
bwrsandman:sdl2-emscripten-non-theaded-fix
Aug 3, 2022
Merged

sdl2: Add patch fixing non-threaded emscripten build#26070
ras0219-msft merged 1 commit into
microsoft:masterfrom
bwrsandman:sdl2-emscripten-non-theaded-fix

Conversation

@bwrsandman
Copy link
Copy Markdown
Contributor

Describe the pull request

  • What does your PR fix?

Fixes regression introduced by changeset 5e7cbdc where emscripten builds of sdl2 that do not use threading will fail during linking time with

error: undefined symbol: emscripten_sync_run_in_main_runtime_thread_ (referenced by top-level compiled C/C++ code)

Source of the fix here: libsdl-org/SDL#5773

  • Which triplets are supported/not supported? Have you updated the CI baseline?

    emscripten, Yes

  • Does your PR follow the maintainer guide?

    Yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    Yes

@Cheney-W Cheney-W self-assigned this Aug 1, 2022
@Cheney-W Cheney-W added category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. labels Aug 1, 2022
@ras0219-msft ras0219-msft merged commit 6a7b3e1 into microsoft:master Aug 3, 2022
@ras0219-msft
Copy link
Copy Markdown
Contributor

LGTM, thanks!

@bwrsandman bwrsandman deleted the sdl2-emscripten-non-theaded-fix branch August 3, 2022 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-bug The issue is with a library, which is something the port should already support info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants