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

[LTS only] Fix synchronization of -java-output-versions with JVM backend #22403

Merged
merged 1 commit into from
Jan 18, 2025

Conversation

WojciechMazur
Copy link
Contributor

Fixes a bug in Scala settings after #22111 which edited wrong ScalaSettings.scala file - the backport to LTS updated max JDK version in the pos/compiler-with-cc tests directory instead of the one in the main sources.

The fix would follow a Scala 3.3.5-RC3 release on Monday

@WojciechMazur WojciechMazur changed the title Fix synchronization of -java-output-versions with JVM backend [LTS only] Fix synchronization of -java-output-versions with JVM backend Jan 17, 2025
@WojciechMazur WojciechMazur merged commit 363970e into scala:release-3.3.5 Jan 18, 2025
20 checks passed
@WojciechMazur WojciechMazur deleted the fix/jdk-23 branch January 18, 2025 10:22
@He-Pin
Copy link

He-Pin commented Jan 18, 2025

@WojciechMazur is Pekko included in Scala 3.6.x's community build? Seems it can't be compiled for now, and I think the community build should add the latest Java version to the matrix too.

@WojciechMazur
Copy link
Contributor Author

Pekko is covered by OpenCB, but was failing for some time now, due to cyclic reference error - I tried at least twice to minimize this issue, but failed each time - here's link to example failing build logs
OpenCB is currently only using JDK LTS releases: 8, 11, 17, 21, we might consider adding support for additional one (latest) in the future, but we'd prefer using only LTS releases due to infrastructure limits.

Regarding managed community build (executed in each PR in scala/scala3 repo) apache/pekko is not covered, however some older revision of akka/akka is tested. It's also limited to usage of JDK 8.

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.

2 participants