Merge branch 'main' into dependabot/github_actions/gradle/gradle-build-action-3.5.0

Author: buenaflor

CHANGELOG.md

@@ -2,14 +2,22 @@
 
 ## Unreleased
 
+### Fixes
+
+- Do not throw if exec operation fails ([#360](https://github.com/getsentry/sentry-kotlin-multiplatform/pull/360))
+
+### Miscellaneous
+
+- Update native android sdk name to `sentry.native.android.kmp` ([#353](https://github.com/getsentry/sentry-kotlin-multiplatform/pull/353))
+
 ### Dependencies
 
-- Bump Cocoa SDK from v8.44.0 to v8.48.0 ([#345](https://github.com/getsentry/sentry-kotlin-multiplatform/pull/345))
-  - [changelog](https://github.com/getsentry/sentry-cocoa/blob/main/CHANGELOG.md#8480)
-  - [diff](https://github.com/getsentry/sentry-cocoa/compare/8.44.0...8.48.0)
-- Bump Java SDK from v7.18.1 to v8.6.0 ([#350](https://github.com/getsentry/sentry-kotlin-multiplatform/pull/350))
-  - [changelog](https://github.com/getsentry/sentry-java/blob/main/CHANGELOG.md#860)
-  - [diff](https://github.com/getsentry/sentry-java/compare/7.18.1...8.6.0)
+- Bump Cocoa SDK from v8.44.0 to v8.49.0 ([#345](https://github.com/getsentry/sentry-kotlin-multiplatform/pull/345), [#363](https://github.com/getsentry/sentry-kotlin-multiplatform/pull/363))
+  - [changelog](https://github.com/getsentry/sentry-cocoa/blob/main/CHANGELOG.md#8490)
+  - [diff](https://github.com/getsentry/sentry-cocoa/compare/8.44.0...8.49.0)
+- Bump Java SDK from v7.18.1 to v8.8.0 ([#350](https://github.com/getsentry/sentry-kotlin-multiplatform/pull/350), [#364](https://github.com/getsentry/sentry-kotlin-multiplatform/pull/364))
+  - [changelog](https://github.com/getsentry/sentry-java/blob/main/CHANGELOG.md#880)
+  - [diff](https://github.com/getsentry/sentry-java/compare/7.18.1...8.8.0)
 
 ## 0.11.0
 

buildSrc/src/main/java/Config.kt

@@ -32,11 +32,11 @@ object Config {
     object Libs {
         val kotlinStd = "org.jetbrains.kotlin:kotlin-stdlib"
 
-        val sentryJavaVersion = "8.6.0"
+        val sentryJavaVersion = "8.8.0"
         val sentryAndroid = "io.sentry:sentry-android:$sentryJavaVersion"
         val sentryJava = "io.sentry:sentry:$sentryJavaVersion"
 
-        val sentryCocoaVersion = "8.48.0"
+        val sentryCocoaVersion = "8.49.0"
         val sentryCocoa = "Sentry"
 
         object Samples {
@@ -83,6 +83,7 @@ object Config {
         val kmpCocoaSdkName = "sentry.cocoa.kmp"
         val kmpJavaSdkName = "sentry.java.kmp"
         val kmpAndroidSdkName = "sentry.java.android.kmp"
+        val kmpNativeAndroidSdkName = "sentry.native.android.kmp"
         val javaPackageName = "maven:io.sentry:sentry"
         val androidPackageName = "maven:io.sentry:sentry-android"
         val cocoaPackageName = "cocoapods:sentry-cocoa"

sentry-kotlin-multiplatform-gradle-plugin/gradle.properties

@@ -2,7 +2,7 @@ id=io.sentry.kotlin.multiplatform.gradle
 implementationClass=io.sentry.kotlin.multiplatform.gradle.SentryPlugin
 versionName=0.11.0
 group=io.sentry
-sentryCocoaVersion=8.48.0
+sentryCocoaVersion=8.49.0
 
 # publication pom properties
 POM_NAME=Sentry Kotlin Multiplatform Gradle Plugin

sentry-kotlin-multiplatform-gradle-plugin/src/main/java/io/sentry/kotlin/multiplatform/gradle/DerivedDataPathValueSource.kt

@@ -41,6 +41,7 @@ abstract class DerivedDataPathValueSource :
             )
             it.standardOutput = buildDirOutput
             it.errorOutput = errOutput
+            it.isIgnoreExitValue = true
         }
 
         if (execOperations.exitValue == 0) {

sentry-kotlin-multiplatform-gradle-plugin/src/main/java/io/sentry/kotlin/multiplatform/gradle/ManualFrameworkPathSearchValueSource.kt

@@ -50,6 +50,7 @@ abstract class ManualFrameworkPathSearchValueSource :
             )
             it.standardOutput = stdOutput
             it.errorOutput = errOutput
+            it.isIgnoreExitValue = true
         }
 
         val stringOutput = stdOutput.toString("UTF-8")

sentry-kotlin-multiplatform/build.gradle.kts

@@ -204,6 +204,7 @@ buildkonfig {
         buildConfigField(STRING, "SENTRY_KMP_COCOA_SDK_NAME", Config.Sentry.kmpCocoaSdkName)
         buildConfigField(STRING, "SENTRY_KMP_JAVA_SDK_NAME", Config.Sentry.kmpJavaSdkName)
         buildConfigField(STRING, "SENTRY_KMP_ANDROID_SDK_NAME", Config.Sentry.kmpAndroidSdkName)
+        buildConfigField(STRING, "SENTRY_KMP_NATIVE_ANDROID_SDK_NAME", Config.Sentry.kmpNativeAndroidSdkName)
 
         buildConfigField(STRING, "VERSION_NAME", project.version.toString())
         buildConfigField(STRING, "SENTRY_JAVA_PACKAGE_NAME", Config.Sentry.javaPackageName)

sentry-kotlin-multiplatform/sentry_kotlin_multiplatform.podspec

@@ -12,7 +12,7 @@ Pod::Spec.new do |spec|
     spec.osx.deployment_target    = '10.13'
     spec.tvos.deployment_target    = '11.0'
     spec.watchos.deployment_target    = '4.0'
-    spec.dependency 'Sentry', '8.48.0'
+    spec.dependency 'Sentry', '8.49.0'
 
     if !Dir.exist?('build/cocoapods/framework/sentry_kotlin_multiplatform.framework') || Dir.empty?('build/cocoapods/framework/sentry_kotlin_multiplatform.framework')
         raise "
@@ -54,4 +54,4 @@ Pod::Spec.new do |spec|
         }
     ]
 
-end
+end

sentry-kotlin-multiplatform/src/androidMain/kotlin/io/sentry/kotlin/multiplatform/SentryPlatformOptions.android.kt

@@ -10,6 +10,7 @@ internal actual fun SentryPlatformOptions.prepareForInit() {
     if (sdkVersion?.packageSet?.none { it.name == BuildKonfig.SENTRY_ANDROID_PACKAGE_NAME } == true) {
         sdkVersion?.addPackage(BuildKonfig.SENTRY_ANDROID_PACKAGE_NAME, BuildKonfig.SENTRY_ANDROID_VERSION)
     }
+    nativeSdkName = BuildKonfig.SENTRY_KMP_NATIVE_ANDROID_SDK_NAME
 }
 
 internal actual fun SentryOptions.toPlatformOptionsConfiguration(): PlatformOptionsConfiguration =

sentry-kotlin-multiplatform/src/androidUnitTest/kotlin/io/sentry/kotlin/multiplatform/SentryAndroidBridgeTest.kt

@@ -0,0 +1,36 @@
+package io.sentry.kotlin.multiplatform
+
+import io.sentry.kotlin.multiplatform.fakes.FakeSentryInstance
+import junit.framework.TestCase.assertEquals
+import kotlin.test.BeforeTest
+import kotlin.test.Test
+
+class SentryAndroidBridgeTest {
+    private lateinit var fixture: Fixture
+
+    @BeforeTest
+    fun setup() {
+        fixture = Fixture()
+    }
+
+    @Test
+    fun `init sets native android sdk name`() {
+        val sut = fixture.getSut()
+
+        sut.init { }
+
+        val option = SentryPlatformOptions().apply {
+            fixture.sentryInstance.lastConfiguration?.invoke(this)
+        }
+
+        assertEquals(BuildKonfig.SENTRY_KMP_NATIVE_ANDROID_SDK_NAME, option.nativeSdkName)
+    }
+}
+
+internal class Fixture {
+    val sentryInstance = FakeSentryInstance()
+
+    fun getSut(): SentryBridge {
+        return SentryBridge(sentryInstance)
+    }
+}

sentry-samples/kmp-app-cocoapods/iosApp/Podfile.lock

@@ -1,9 +1,9 @@
 PODS:
-  - Sentry (8.48.0):
-    - Sentry/Core (= 8.48.0)
-  - Sentry/Core (8.48.0)
+  - Sentry (8.49.0):
+    - Sentry/Core (= 8.49.0)
+  - Sentry/Core (8.49.0)
   - shared (1.0):
-    - Sentry (= 8.48.0)
+    - Sentry (= 8.49.0)
 
 DEPENDENCIES:
   - shared (from `../shared`)
@@ -17,8 +17,8 @@ EXTERNAL SOURCES:
     :path: "../shared"
 
 SPEC CHECKSUMS:
-  Sentry: 1ca8405451040482877dcd344dfa3ef80b646631
-  shared: a39a59e646f40aca21465f5b57c0ba28df8aaf8e
+  Sentry: 5eda2559a20fa0c377b695ed9d7c178a47122891
+  shared: 1b9a92de3fc68a6ae8f1d23b53e3a607cd5cdff7
 
 PODFILE CHECKSUM: f282da88f39e69507b0a255187c8a6b644477756