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.

Sign up for GitHub

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

Jump to bottom

Set Xcode version #2090

Merged
merged 2 commits into from
Dec 16, 2022
Merged

Set Xcode version #2090

merged 2 commits into from
Dec 16, 2022

Conversation

mattjohnsonpint
Copy link
Contributor

@mattjohnsonpint mattjohnsonpint commented Dec 16, 2022 •
edited
Loading

Fix macOS / iOS build failures due to default Xcode being 14.0.1 but MAUI needing 14.1

See https://github.com/actions/runner-images/blob/main/images/macos/macos-12-Readme.md#xcode

Also: https://github.com/getsentry/sentry-dotnet/actions/runs/3709206020/jobs/6287586423#step:8:616

ILLINK : error MT2301: The linker step 'Setup' failed during processing: This version of Microsoft.iOS requires the iOS 16.1 SDK (shipped with Xcode 14.1). Either upgrade Xcode to get the required header files or set the managed linker behaviour to Link Framework SDKs Only in your project's iOS Build Options > Linker Behavior (to try to avoid the new APIs). [/Users/runner/work/sentry-dotnet/sentry-dotnet/samples/Sentry.Samples.Ios/Sentry.Samples.Ios.csproj]

#skip-changelog

@mattjohnsonpint mattjohnsonpint marked this pull request as draft December 16, 2022 03:45
@mattjohnsonpint
Copy link
Contributor Author

Reference issue: actions/runner-images#6771

@mattjohnsonpint mattjohnsonpint marked this pull request as ready for review December 16, 2022 04:15
@mattjohnsonpint mattjohnsonpint enabled auto-merge (squash) December 16, 2022 04:16
@mattjohnsonpint mattjohnsonpint mentioned this pull request Dec 16, 2022
Merged
@mattjohnsonpint mattjohnsonpint merged commit 2b3ce10 into main Dec 16, 2022
@mattjohnsonpint mattjohnsonpint deleted the fix-xcode branch December 16, 2022 13:58
@mattjohnsonpint mattjohnsonpint mentioned this pull request Jan 24, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bruno-garcia bruno-garcia bruno-garcia approved these changes

@bitsandfoxes bitsandfoxes Awaiting requested review from bitsandfoxes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mattjohnsonpint @bruno-garcia