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.

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

c, obj-c and cpp samples #342

Merged
merged 7 commits into from
Sep 28, 2021
Merged

c, obj-c and cpp samples #342

merged 7 commits into from
Sep 28, 2021

Conversation

bruno-garcia
Copy link
Member

@bruno-garcia bruno-garcia commented Sep 27, 2021

In both new samples I wasn't able to get a symbolicated stack trace properly. Even though I ran sentry-cli on the directory I configured Unity to create the xcode project and also on the original Unity project (so it can get anything under Temp). In both situations I got:

image

It could symbolicate frames from the SDK itself (since the .framework is copied to the xcode project and sentry-cli found it there) but I didn't find the UnityFramework.framework.

Sounds like it's something we need to find out and properly document so users don't get confused.

In any case, here we have 2 more samples: C++ Exceptions and C++ segfault

The Objective-C example requires an additional configuration which requires documenting: #343

@bruno-garcia bruno-garcia changed the title c and cpp example c, obj-c and cpp samples Sep 27, 2021
CHANGELOG.md Outdated Show resolved Hide resolved
@@ -170,8 +170,8 @@ MonoBehaviour:
m_Enabled: 1
m_EditorHideFlags: 0
m_Script: {fileID: 11500000, guid: 4e29b1a8efbd4b44bb3f3716e73f07ff, type: 3}
m_Name:
m_EditorClassIdentifier:
m_Name:
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure what caused all these white spaces. Tricky to revert the file while keeping the new buttons only.

@bruno-garcia bruno-garcia merged commit aec19be into main Sep 28, 2021
@bruno-garcia bruno-garcia deleted the sample/native branch September 28, 2021 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants