Skip to content

Commit c2af820

Browse files
authored
Update to Gradle 8.8 (#5977)
* Update to Gradle 8.8 * update wrapepr
1 parent 8d609cd commit c2af820

File tree

3 files changed

+18
-15
lines changed

3 files changed

+18
-15
lines changed

build-logic/src/main/kotlin/CompilerOptions.kt

Lines changed: 16 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,16 @@ import org.jetbrains.kotlin.gradle.tasks.KotlinCompilationTask
2222
/**
2323
* @param target the JVM version we want to be compatible with (bytecode + bootstrap classpath)
2424
*/
25-
fun KotlinCommonCompilerOptions.configure(target: Int, kotlinCompilerOptions: KotlinCompilerOptions) {
25+
fun KotlinCommonCompilerOptions.configure(target: Int, kotlinCompilerOptions: KotlinCompilerOptions, isAndroid: Boolean) {
26+
val actualTarget = when {
27+
isAndroid -> {
28+
// https://blog.blundellapps.co.uk/setting-jdk-level-in-android-gradle-builds/
29+
// D8 can dex Java17 bytecode
30+
17
31+
}
32+
else -> target
33+
}
34+
2635
freeCompilerArgs.add("-Xexpect-actual-classes")
2736

2837
/**
@@ -39,8 +48,11 @@ fun KotlinCommonCompilerOptions.configure(target: Int, kotlinCompilerOptions: Ko
3948
when (this) {
4049
is KotlinJvmCompilerOptions -> {
4150
freeCompilerArgs.add("-Xjvm-default=all")
42-
freeCompilerArgs.add("-Xjdk-release=${target.toJvmTarget().target}")
43-
jvmTarget.set(target.toJvmTarget())
51+
if (!isAndroid) {
52+
// See https://cs.android.com/android-studio/platform/tools/base/+/mirror-goog-studio-main:build-system/gradle-core/src/main/java/com/android/build/gradle/tasks/JavaCompileUtils.kt;l=410?q=Using%20%27--release%27%20option%20for%20JavaCompile%20is%20not%20supported%20because%20it%20prevents%20the%20Android%20Gradle%20plugin
53+
freeCompilerArgs.add("-Xjdk-release=${actualTarget.toJvmTarget().target}")
54+
}
55+
jvmTarget.set(actualTarget.toJvmTarget())
4456
}
4557

4658
is KotlinNativeCompilerOptions -> {
@@ -98,16 +110,7 @@ fun Project.configureJavaAndKotlinCompilers(jvmTarget: Int?, kotlinCompilerOptio
98110
val jvmTarget = jvmTarget?: 8
99111

100112
kotlinExtensionOrNull?.forEachCompilerOptions { isAndroid ->
101-
val target = when {
102-
isAndroid -> {
103-
// https://blog.blundellapps.co.uk/setting-jdk-level-in-android-gradle-builds/
104-
// D8 can dex Java17 bytecode
105-
17
106-
}
107-
else -> jvmTarget
108-
}
109-
110-
configure(target, kotlinCompilerOptions)
113+
configure(jvmTarget, kotlinCompilerOptions, isAndroid)
111114
}
112115
project.tasks.withType(JavaCompile::class.java).configureEach {
113116
// For JVM only modules, this dictates the "org.gradle.jvm.version" Gradle attribute

gradle/wrapper/gradle-wrapper.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
distributionBase=GRADLE_USER_HOME
22
distributionPath=wrapper/dists
3-
distributionUrl=https\://services.gradle.org/distributions/gradle-8.7-rc-1-bin.zip
3+
distributionUrl=https\://services.gradle.org/distributions/gradle-8.8-bin.zip
44
networkTimeout=10000
55
validateDistributionUrl=true
66
zipStoreBase=GRADLE_USER_HOME

gradlew

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@
5555
# Darwin, MinGW, and NonStop.
5656
#
5757
# (3) This script is generated from the Groovy template
58-
# https://github.com/gradle/gradle/blob/HEAD/subprojects/plugins/src/main/resources/org/gradle/api/internal/plugins/unixStartScript.txt
58+
# https://github.com/gradle/gradle/blob/HEAD/platforms/jvm/plugins-application/src/main/resources/org/gradle/api/internal/plugins/unixStartScript.txt
5959
# within the Gradle project.
6060
#
6161
# You can find Gradle at https://github.com/gradle/gradle/.

0 commit comments

Comments
 (0)