ci: fix/set MtouchUseLlvm=false to speed up the macOS build
#4677
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reduces the CI round-trip time from 2+ hours to less than 1 hour.
Samples
The iOS target platform check in
samples/Directory.Build.propsdid not work as expected.Sentry.Samples.IosandSentry.Samples.Mauiwere built withMtouchUseLlvm=true. We can remove the redundant iOS target platform check because the property is only applicable to iOS, tvOS, and Mac Catalyst, and never enabled by default on Mac Catalyst.https://learn.microsoft.com/en-us/dotnet/ios/building-apps/build-properties#mtouchusellvm
Tests
The property was already set for
Sentry.Maui.Device.IntegrationTestApp. Now, it is also set forSentry.Maui.Device.TestAppand the dynamically createdmaui-appintegration test.See also