-
-
Notifications
You must be signed in to change notification settings - Fork 216
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.
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
4.12: SentryMauiOptions.Native not available on IOS anymore (was ok on 4.11) #3648
Comments
@softlion the Native options are used in the sample apps, e.g. here:
So the solution wouldn't compile and we couldn't make a release if these weren't available. Possibly clean your solution and restore workloads/packages? |
I'm experiencing the same thing. Had to downgrade. Even fails in CI where the caches start out from scratch. |
I have the same issue too. It work's on the sample project because sentry is referenced using a project reference. If you replace it using the NuGet package, the compilation will failed. |
Diff: 4.11.0...4.12.0 We didn't update The only thing related to iOS that changed, was the xcode version. Went from xcode 15.4 to 16 I wonder if the .NET tooling has some issues with this version of xcode. |
I'm unable to reproduce this. Here's my
If anyone can provide a minimal reproduction then I can look into this though. |
And using DotPeek v2024.2.5 I have a hard time finding the existence of the property Native as well as its type NativeOptions. In this case for both Android and iOS assemblies. And both v4.11 and v4.12. Something is not right. For v4.11 it at least works in Github CI and Visual Studio / ReSharper. |
I see there are issues with MAUI and XCode 16. Might be related to this issue as Sentry v4.12 started using XCode 16. |
And there are different targets between 4.11 and 4.12 in https://www.nuget.org/packages/Sentry. I guess not everyone is building with the latest MAUI workload, which is true in my case. CI is locked to an older MAUI workload for iOS due to bugs introduces in the latest one. |
You found the issue !! To use this mew sentry nuget we all should use the latest net SDK which is 18 and requires xcode 16. Sentry devs could instead specify both net8.0-ios and net8.0-ios17.5 in the targets. |
Aha, that makes sense. That was the main reason for making the 4.12 release (to support XCode 16).
I'm not sure if that's practical. The reason we bumped to We'll discuss internally but for the time being I'll put a big warning about the change in dependencies in the change log, so hopefully fewer people get caught out by this. Apologies in advance for any trouble this has caused folks. We'll definitely look into what happened here and try to avoid it happening again in future. Changes to tooling from multiple vendors who don't coordinate doesn't help things, but that's the world we live in so we need some good checks and balances in place to mitigate the impact of that on Sentry SDK users. |
Package
Sentry
.NET Flavor
.NET
.NET Version
8.0.0
OS
iOS
SDK Version
4.12
Self-Hosted Sentry Version
No response
Steps to Reproduce
Expected Result
No error building this code.
This code does compile ok on 4.11
Actual Result
The
Native
property is not available anymore.The text was updated successfully, but these errors were encountered: