Skip to content

Conversation

@github-actions
Copy link
Contributor

Bumps scripts/update-java.ps1 from 8.21.0 to 8.21.1.

Auto-generated by a dependency updater.

Changelog

8.21.1

Fixes

  • Use Kotlin stdlib 1.9.24 dependency instead of 2.2.0 for all Android modules (#4707)
    • This fixes compile time issues if your app is using Kotlin < 2.x

@github-actions github-actions bot added the Dependencies Pull requests that update a dependency file label Sep 10, 2025
@bruno-garcia bruno-garcia force-pushed the deps/scripts/update-java.ps1 branch from 671d23e to 06e2af0 Compare September 10, 2025 03:08
@jpnurmi
Copy link
Collaborator

jpnurmi commented Sep 10, 2025

I was too quick to bump the Kotlin version in #4502. Probably better to revert from 2.2.0 (June 23, 2025) back to 2.0.0 (May 21, 2024) right?

Copy link
Collaborator

@jamescrosswell jamescrosswell left a comment

Choose a reason for hiding this comment

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

LGTM

@jamescrosswell jamescrosswell self-requested a review September 10, 2025 09:10
@jamescrosswell
Copy link
Collaborator

I was too quick to bump the Kotlin version in #4502. Probably better to revert from 2.2.0 (June 23, 2025) back to 2.0.0 (May 21, 2024) right?

Ah, I had assumed we had to bump this in order to use the latest Sentry SDK for Java... ideally we'd target the oldest possible version we can get away with (so as not to force our customers to have to bump their versions in order to use Sentry).

@jpnurmi
Copy link
Collaborator

jpnurmi commented Sep 10, 2025

Yeah, for sentry-java 8.21.0 we had to bump the Kotlin version, but they hot-fixed it in 8.21.1:

Is it too late to include this in sentry-dotnet 5.15.0?

@jamescrosswell
Copy link
Collaborator

jamescrosswell commented Sep 10, 2025

Is it too late to include this in sentry-dotnet 5.15.0?

I don't think that release has gone out yet... maybe coordinate with @Flash0ver (I'm just about to go to sleep) if you want to cancel it and roll it out again. Otherwise, we could always make a 5.15.1 release.

@Flash0ver
Copy link
Member

Flash0ver commented Sep 10, 2025

I prefer to retract the currently open 5.15.0 release, because it does not include the bump of

-<PackageReference Include="Xamarin.Kotlin.StdLib.Jdk8" Version="2.0.0" />
+<PackageReference Include="Xamarin.Kotlin.StdLib.Jdk8" Version="2.2.0" />

yet, rather than decrementing the dependency later in a patch-release again.

@jamescrosswell
Copy link
Collaborator

Yeah, for sentry-java 8.21.0 we had to bump the Kotlin version, but they hot-fixed it in 8.21.1:

I've retracted the 5.15.0 release... still not sure I follow what's going on with the versions above though. It looks like the Java SDK only depends on version 1.9.24, so why do we need to bump from version 2.0.0 to 2.2.0?

@Flash0ver Flash0ver merged commit 64de9b6 into main Sep 10, 2025
32 checks passed
@Flash0ver Flash0ver deleted the deps/scripts/update-java.ps1 branch September 10, 2025 22:40
@jpnurmi
Copy link
Collaborator

jpnurmi commented Sep 11, 2025

There's no version 1.9.24 of the Xamarin.Kotlin.StdLib NuGet package. We use the nearest available version 2.0.0. The dependency was originally added in:

When bumping sentry-java to 8.21.0 we had to bump Kotlin to 2.2.0 due to:

Error: error XA4242: Java dependency 'org.jetbrains.kotlin:kotlin-stdlib-jdk8:2.2.0' is not satisfied.

The dependency on Kotlin 2.2 was reverted in sentry-java 8.21.1:

=> We reverted to 2.0.0.

@jamescrosswell
Copy link
Collaborator

=> We reverted to 2.0.0.

Ah, I see. Great - thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants