Skip to content

build: allow local modules/sentry-cocoa clone for development #4551

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

Merged
jpnurmi merged 7 commits into from
Oct 2, 2025
Merged

build: allow local modules/sentry-cocoa clone for development #4551

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
0d4e77f
build: allow local `modules/sentry-cocoa` clone for development
jpnurmi Sep 22, 2025
ae1e606
Drop redundant !Exists('$(SentryCocoaFramework)') checks
jpnurmi Sep 22, 2025
03e08fb
Update modules/README.md
jpnurmi Sep 23, 2025
0bc1850
Fix path to .git\HEAD
jpnurmi Sep 23, 2025
05bd4cf
Fix _BuildCocoaSDK and _GenerateSentryCocoaBindings inputs
jpnurmi Sep 24, 2025
59da77d
Move instructions to CONTRIBUTING.md
jpnurmi Sep 29, 2025
9c7a198
Remove arm64e
jpnurmi Oct 2, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update modules/README.md
  • Loading branch information
@jpnurmi
jpnurmi committed Sep 23, 2025
commit 03e08fb7bfc544b04e00201287ee212f27e37687
10 changes: 10 additions & 0 deletions modules/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,3 +15,13 @@ To make sure we're not exposing any API from that library externally.
Fork [getsentry/perfview](https://github.com/getsentry/perfview/).
Tool from the .NET team which includes several utilities used for profiling .NET code.
We use that in our `Sentry.Profiling` package.

Copy link
Member

@Flash0ver Flash0ver Sep 24, 2025

Choose a reason for hiding this comment

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

I believe we also miss an entry for sentry-native ... but that's not part of this changeset ... I can follow-up on that afterwards.

### sentry-cocoa

By default, `Sentry.Bindings.Cocoa` downloads a pre-built Sentry Cocoa SDK from
GitHub Releases. The version is specified in `sentry-cocoa.properties`.

If you want to build an unreleased Sentry Cocoa SDK version from source instead,
replace the pre-built SDK with [getsentry/sentry-cocoa](https://github.com/getsentry/sentry-cocoa/)
by cloning it into the `modules/sentry-cocoa` directory. To switch back to the
pre-built SDK, delete the `modules/sentry-cocoa` directory.
Loading