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

Update documentation to make it clearer CocoaPods is not mandatory #298

Open
tfcporciuncula opened this issue Nov 28, 2024 · 2 comments
Open

Comments

@tfcporciuncula
Copy link

Problem Statement

Currently, the docs on https://docs.sentry.io/platforms/kotlin-multiplatform suggest CocoaPods is mandatory when integrating the library. There's even a section on SPM at the bottom that says we can use SPM for other things, but we still need CocoaPods for Sentry.

Is this really true, though? We tried adding Sentry directly on iOS with SPM (https://docs.sentry.io/platforms/apple/install/swift-package-manager) and it seems to have worked fine. Looking at #118 and #108 it also seems to confirm this strategy would be fine.

It would be great to get clarity on this and get the docs updated if they're indeed misleading.

Solution Brainstorm

No response

@buenaflor
Copy link
Contributor

I assume your is Static = true is set in the framework? in that case it should work but not for tests because the linker is not configured there, that's the main issue. When you look at the cocoapods plugin source code they set up the linking automatically that's why it's suggested to set up both. If you don't setup linking you will get this error

Regardless we actually have new docs merged in today (not sure when they will be live) that suggests to use our gradle plugin that does all the linking so you're not tied to using only cocoapods. Please try it out

plugins { id("io.sentry.kotlin.multiplatform.gradle") version "0.10.0" }

@tfcporciuncula
Copy link
Author

Yes, we do set isStatic = true! And thanks, we'll try out the plugin and report back.

@buenaflor buenaflor moved this from Needs Discussion to Needs Investigation in Mobile & Cross Platform SDK Dec 4, 2024
@buenaflor buenaflor moved this from Needs Investigation to Needs More Information in Mobile & Cross Platform SDK Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Needs More Information
Development

No branches or pull requests

2 participants