iOS native support #1727
iOS native support #1727
Conversation
bruno-garcia
requested review from
SimonCropp and
mattjohnsonpint
as code owners
Instructions and example for changelogPlease add an entry to Example: - iOS native support ([#1727](https://github.com/getsentry/sentry-dotnet/pull/1727))If none of the above apply, you can opt out of this check by adding |
…os-bindings #the commit.
bruno-garcia
force-pushed
the
feat/ios-bindings
branch
from
4e754a1
to
ee90f56
Compare
| @@ -5,7 +5,7 @@ | |||
| <!-- BG8605 and BG8606 happen because there's a missing androidx.lifecycle dependency, but we don't need it here. (The native Android Sentry SDK will use it if it exists.) --> | |||
| <NoWarn>$(NoWarn);BG8605;BG8606</NoWarn> | |||
| <SentryAndroidSdkVersion>6.0.0</SentryAndroidSdkVersion> | |||
| <AndroidLibsDirectory>$(BaseIntermediateOutputPath)\libs\Android\</AndroidLibsDirectory> | |||
| <AndroidLibsDirectory>$(BaseIntermediateOutputPath)\SentrySDKs\Android\$(SentryAndroidSdkVersion)\</AndroidLibsDirectory> | |||
There was a problem hiding this comment.
Less relevant on Android because artifacts have the version on the file name, but puts things under the version to avoid issues when we update:
|
Blocked by: dotnet/msbuild#4584
|
|
Blocked on: xamarin/xamarin-macios#15299 |
| // o.Debug = options.Debug; | ||
| // }); | ||
|
|
||
| SentrySDK.CaptureMessage("Testing binding to iOS"); |
There was a problem hiding this comment.
Works (this event came from sentry-cocoa):
Stack trace looks like that because we don't upload symbols yet on build
| EventNotSent = 107 | ||
| } | ||
|
|
||
| //static class CFunctions |
There was a problem hiding this comment.
why the commented out code?
| @@ -0,0 +1,1273 @@ | |||
| using System; | |||
There was a problem hiding this comment.
is this file code-gen'd?
There was a problem hiding this comment.
yes, Objective Sharpie
|
Closing in favor of #1829. I used a lot of this though. Thanks!!! |
Hopefully one day: Resolves #1282