Skip to content

[sdl2] Fix pthread detection on macOS#17106

Merged
vicroms merged 4 commits intomicrosoft:masterfrom
past-due:sdl2_mac_cross_compile_1
Apr 7, 2021
Merged

[sdl2] Fix pthread detection on macOS#17106
vicroms merged 4 commits intomicrosoft:masterfrom
past-due:sdl2_mac_cross_compile_1

Conversation

@past-due
Copy link
Contributor

@past-due past-due commented Apr 5, 2021

This fixes cross-compiling the sdl2 port on macOS.

Specifically, in a common CI scenario, an x64 build machine is used to build an arm64 version of the port. Previously, this would fail to properly detect the availability of pthreads, and would result in an arm64 build of sdl2 that was (quietly) missing support for SDL's "Threads" subsystem. (Which would likely only be discovered when attempting to run a dependent program on arm64 macOS, which would then fail out.)

Also includes a small drive-by fix for a deprecation warning re: vcpkg_check_features.

@NancyLi1013 NancyLi1013 self-assigned this Apr 6, 2021
@NancyLi1013 NancyLi1013 added the category:port-bug The issue is with a library, which is something the port should already support label Apr 6, 2021
@NancyLi1013 NancyLi1013 added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Apr 6, 2021
@vicroms vicroms merged commit b3e8d4d into microsoft:master Apr 7, 2021
@past-due past-due deleted the sdl2_mac_cross_compile_1 branch November 27, 2021 00:35
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