Skip to content

Commit d756bf4

Browse files
wilkinsonawilkinsona
committed
Upgrade build to Gradle 8.10
Closes gh-41980
1 parent 275d366 commit d756bf4

File tree

24 files changed

+99
-94
lines changed

24 files changed

+99
-94
lines changed

‎.github/workflows/ci.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,12 +22,12 @@ jobs:
2222
- version: 17
2323
toolchain: false
2424
- version: 21
25-
toolchain: true
25+
toolchain: false
2626
- version: 22
27-
toolchain: true
27+
toolchain: false
2828
- version: 23
2929
early-access: true
30-
toolchain: true
30+
toolchain: false
3131
exclude:
3232
- os:
3333
name: Linux

‎buildSrc/build.gradle

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,11 @@ repositories {
1010
gradlePluginPortal()
1111
}
1212

13-
sourceCompatibility = 17
14-
targetCompatibility = 17
13+
java {
14+
sourceCompatibility = 17
15+
targetCompatibility = 17
16+
}
17+
1518

1619
def versions = [:]
1720
new File(projectDir.parentFile, "gradle.properties").withInputStream {

‎buildSrc/src/main/java/org/springframework/boot/build/KotlinConventions.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ class KotlinConventions {
5151
void apply(Project project) {
5252
project.getPlugins().withId("org.jetbrains.kotlin.jvm", (plugin) -> {
5353
project.getTasks().withType(KotlinCompile.class, this::configure);
54-
configureDokkatoo(project);
54+
project.getPlugins().withType(DokkatooHtmlPlugin.class, (dokkatooPlugin) -> configureDokkatoo(project));
5555
});
5656
}
5757

@@ -67,7 +67,6 @@ private void configure(KotlinCompile compile) {
6767
}
6868

6969
private void configureDokkatoo(Project project) {
70-
project.getPlugins().apply(DokkatooHtmlPlugin.class);
7170
DokkatooExtension dokkatoo = project.getExtensions().getByType(DokkatooExtension.class);
7271
dokkatoo.getDokkatooSourceSets().named(SourceSet.MAIN_SOURCE_SET_NAME).configure((sourceSet) -> {
7372
sourceSet.getSourceRoots().setFrom(project.file("src/main/kotlin"));

‎buildSrc/src/main/java/org/springframework/boot/build/bom/bomr/InteractiveUpgradeResolver.java

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2012-2023 the original author or authors.
2+
* Copyright 2012-2024 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -16,6 +16,7 @@
1616

1717
package org.springframework.boot.build.bom.bomr;
1818

19+
import java.util.ArrayList;
1920
import java.util.Collection;
2021
import java.util.HashMap;
2122
import java.util.List;
@@ -58,12 +59,17 @@ private Upgrade resolveUpgrade(LibraryWithVersionOptions libraryWithVersionOptio
5859
if (libraryWithVersionOptions.getVersionOptions().isEmpty()) {
5960
return null;
6061
}
61-
VersionOption current = new VersionOption(libraryWithVersionOptions.getLibrary().getVersion().getVersion());
62-
VersionOption selected = this.userInputHandler.selectOption(
63-
libraryWithVersionOptions.getLibrary().getName() + " "
64-
+ libraryWithVersionOptions.getLibrary().getVersion().getVersion(),
65-
libraryWithVersionOptions.getVersionOptions(), current);
66-
return (selected.equals(current)) ? null
62+
VersionOption defaultOption = new VersionOption(
63+
libraryWithVersionOptions.getLibrary().getVersion().getVersion());
64+
VersionOption selected = this.userInputHandler.askUser((questions) -> {
65+
String question = libraryWithVersionOptions.getLibrary().getName() + " "
66+
+ libraryWithVersionOptions.getLibrary().getVersion().getVersion();
67+
List<VersionOption> options = new ArrayList<>();
68+
options.add(defaultOption);
69+
options.addAll(libraryWithVersionOptions.getVersionOptions());
70+
return questions.selectOption(question, options, defaultOption);
71+
}).get();
72+
return (selected.equals(defaultOption)) ? null
6773
: new Upgrade(libraryWithVersionOptions.getLibrary(), selected.getVersion());
6874
}
6975

‎buildSrc/src/main/java/org/springframework/boot/build/test/DockerTestPlugin.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,8 @@ private void configureDockerTesting(Project project) {
7171
.add(project.getConfigurations()
7272
.getByName(dockerTestSourceSet.getRuntimeClasspathConfigurationName())));
7373
});
74+
project.getDependencies()
75+
.add(dockerTestSourceSet.getRuntimeOnlyConfigurationName(), "org.junit.platform:junit-platform-launcher");
7476
}
7577

7678
private SourceSet createSourceSet(Project project) {

‎buildSrc/src/main/java/org/springframework/boot/build/test/IntegrationTestPlugin.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2012-2023 the original author or authors.
2+
* Copyright 2012-2024 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -58,6 +58,8 @@ private void configureIntegrationTesting(Project project) {
5858
eclipse.classpath((classpath) -> classpath.getPlusConfigurations()
5959
.add(project.getConfigurations().getByName(intTestSourceSet.getRuntimeClasspathConfigurationName())));
6060
});
61+
project.getDependencies()
62+
.add(intTestSourceSet.getRuntimeOnlyConfigurationName(), "org.junit.platform:junit-platform-launcher");
6163
}
6264

6365
private SourceSet createSourceSet(Project project) {

‎gradle/wrapper/gradle-wrapper.jar

121 Bytes
Binary file not shown.

‎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-7.6.4-bin.zip
3+
distributionUrl=https\://services.gradle.org/distributions/gradle-8.10-bin.zip
44
networkTimeout=10000
55
validateDistributionUrl=true
66
zipStoreBase=GRADLE_USER_HOME

‎gradlew

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@
1515
# See the License for the specific language governing permissions and
1616
# limitations under the License.
1717
#
18+
# SPDX-License-Identifier: Apache-2.0
19+
#
1820

1921
##############################################################################
2022
#
@@ -55,7 +57,7 @@
5557
# Darwin, MinGW, and NonStop.
5658
#
5759
# (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
60+
# https://github.com/gradle/gradle/blob/HEAD/platforms/jvm/plugins-application/src/main/resources/org/gradle/api/internal/plugins/unixStartScript.txt
5961
# within the Gradle project.
6062
#
6163
# You can find Gradle at https://github.com/gradle/gradle/.
@@ -84,7 +86,8 @@ done
8486
# shellcheck disable=SC2034
8587
APP_BASE_NAME=${0##*/}
8688
# Discard cd standard output in case $CDPATH is set (https://github.com/gradle/gradle/issues/25036)
87-
APP_HOME=$( cd "${APP_HOME:-./}" > /dev/null && pwd -P ) || exit
89+
APP_HOME=$( cd -P "${APP_HOME:-./}" > /dev/null && printf '%s
90+
' "$PWD" ) || exit
8891

8992
# Use the maximum available, or set MAX_FD != -1 to use that value.
9093
MAX_FD=maximum

‎gradlew.bat

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,8 @@
1313
@rem See the License for the specific language governing permissions and
1414
@rem limitations under the License.
1515
@rem
16+
@rem SPDX-License-Identifier: Apache-2.0
17+
@rem
1618

1719
@if "%DEBUG%"=="" @echo off
1820
@rem ##########################################################################

‎spring-boot-project/spring-boot-actuator-autoconfigure/build.gradle

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -203,6 +203,8 @@ tasks.named("test") {
203203
}
204204

205205
def documentationTest = tasks.register("documentationTest", Test) {
206+
testClassesDirs = testing.suites.test.sources.output.classesDirs
207+
classpath = testing.suites.test.sources.runtimeClasspath
206208
jvmArgs += "--add-opens=java.base/java.net=ALL-UNNAMED"
207209
filter {
208210
includeTestsMatching("org.springframework.boot.actuate.autoconfigure.endpoint.web.documentation.*")

‎spring-boot-project/spring-boot-docs/build.gradle

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
plugins {
2+
id "dev.adamko.dokkatoo-html"
23
id "java"
34
id "org.antora"
45
id "org.springframework.boot.conventions"
@@ -43,7 +44,7 @@ sourcesJar {
4344
}
4445

4546
plugins.withType(EclipsePlugin) {
46-
extensions.getByType(org.gradle.plugins.ide.eclipse.model.EclipseModel).classpath { classpath ->
47+
eclipse.classpath { classpath ->
4748
classpath.plusConfigurations.add(configurations.getByName(sourceSets.main.runtimeClasspathConfigurationName))
4849
}
4950
}

‎spring-boot-project/spring-boot-test/build.gradle

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
plugins {
2+
id "dev.adamko.dokkatoo-html"
23
id "java-library"
34
id "org.jetbrains.kotlin.jvm"
45
id "org.springframework.boot.conventions"

‎spring-boot-project/spring-boot-tools/spring-boot-cli/build.gradle

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,11 +82,11 @@ def configureArchive(archive) {
8282
into "lib/"
8383
}
8484
archive.from(file("src/main/content")) {
85-
dirMode = 0755
86-
fileMode = 0644
85+
dirPermissions { unix(0755) }
86+
filePermissions { unix(0644) }
8787
}
8888
archive.from(file("src/main/executablecontent")) {
89-
fileMode = 0755
89+
filePermissions { unix(0755) }
9090
}
9191
}
9292

‎spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -140,10 +140,6 @@ artifacts {
140140
antoraContent antoraGradlePluginCatalogContent
141141
}
142142

143-
toolchain {
144-
maximumCompatibleJavaVersion = JavaLanguageVersion.of(20)
145-
}
146-
147143
plugins.withType(EclipsePlugin) {
148144
eclipse {
149145
classpath.file { merger ->

‎spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/src/main/java/org/springframework/boot/gradle/plugin/ApplicationPluginAction.java

Lines changed: 7 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,8 @@
1919
import java.io.IOException;
2020
import java.io.InputStreamReader;
2121
import java.io.StringWriter;
22-
import java.lang.reflect.Method;
2322
import java.util.concurrent.Callable;
2423

25-
import org.gradle.api.Action;
2624
import org.gradle.api.GradleException;
2725
import org.gradle.api.Plugin;
2826
import org.gradle.api.Project;
@@ -128,32 +126,16 @@ private String loadResource(String name) {
128126

129127
private void configureFilePermissions(CopySpec copySpec, int mode) {
130128
if (GradleVersion.current().compareTo(GradleVersion.version("8.3")) >= 0) {
131-
try {
132-
Method filePermissions = copySpec.getClass().getMethod("filePermissions", Action.class);
133-
filePermissions.invoke(copySpec, new Action<>() {
134-
135-
@Override
136-
public void execute(Object filePermissions) {
137-
String unixPermissions = Integer.toString(mode, 8);
138-
try {
139-
Method unix = filePermissions.getClass().getMethod("unix", String.class);
140-
unix.invoke(filePermissions, unixPermissions);
141-
}
142-
catch (Exception ex) {
143-
throw new GradleException("Failed to set file permissions to '" + unixPermissions + "'",
144-
ex);
145-
}
146-
}
147-
148-
});
149-
}
150-
catch (Exception ex) {
151-
throw new GradleException("Failed to set file permissions", ex);
152-
}
129+
copySpec.filePermissions((filePermissions) -> filePermissions.unix(Integer.toString(mode, 8)));
153130
}
154131
else {
155-
copySpec.setFileMode(mode);
132+
configureFileMode(copySpec, mode);
156133
}
157134
}
158135

136+
@SuppressWarnings("deprecation")
137+
private void configureFileMode(CopySpec copySpec, int mode) {
138+
copySpec.setFileMode(mode);
139+
}
140+
159141
}

‎spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/src/main/java/org/springframework/boot/gradle/tasks/bundling/BootArchiveSupport.java

Lines changed: 24 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,8 @@
2626
import java.util.Set;
2727
import java.util.TreeMap;
2828
import java.util.function.Function;
29-
import java.util.function.Supplier;
3029

31-
import org.gradle.api.GradleException;
30+
import org.gradle.api.file.ConfigurableFilePermissions;
3231
import org.gradle.api.file.CopySpec;
3332
import org.gradle.api.file.FileCopyDetails;
3433
import org.gradle.api.file.FileTreeElement;
@@ -133,44 +132,44 @@ CopyAction createCopyAction(Jar jar, ResolvedDependencies resolvedDependencies,
133132
File output = jar.getArchiveFile().get().getAsFile();
134133
Manifest manifest = jar.getManifest();
135134
boolean preserveFileTimestamps = jar.isPreserveFileTimestamps();
136-
Integer dirMode = getDirMode(jar);
137-
Integer fileMode = getFileMode(jar);
135+
Integer dirPermissions = getUnixNumericDirPermissions(jar);
136+
Integer filePermissions = getUnixNumericFilePermissions(jar);
138137
boolean includeDefaultLoader = isUsingDefaultLoader(jar);
139138
Spec<FileTreeElement> requiresUnpack = this.requiresUnpack.getAsSpec();
140139
Spec<FileTreeElement> exclusions = this.exclusions.getAsExcludeSpec();
141140
LaunchScriptConfiguration launchScript = this.launchScript;
142141
Spec<FileCopyDetails> librarySpec = this.librarySpec;
143142
Function<FileCopyDetails, ZipCompression> compressionResolver = this.compressionResolver;
144143
String encoding = jar.getMetadataCharset();
145-
CopyAction action = new BootZipCopyAction(output, manifest, preserveFileTimestamps, dirMode, fileMode,
146-
includeDefaultLoader, jarmodeToolsLocation, requiresUnpack, exclusions, launchScript, librarySpec,
147-
compressionResolver, encoding, resolvedDependencies, supportsSignatureFile, layerResolver,
144+
CopyAction action = new BootZipCopyAction(output, manifest, preserveFileTimestamps, dirPermissions,
145+
filePermissions, includeDefaultLoader, jarmodeToolsLocation, requiresUnpack, exclusions, launchScript,
146+
librarySpec, compressionResolver, encoding, resolvedDependencies, supportsSignatureFile, layerResolver,
148147
loaderImplementation);
149148
return jar.isReproducibleFileOrder() ? new ReproducibleOrderingCopyAction(action) : action;
150149
}
151150

152-
private Integer getDirMode(CopySpec copySpec) {
153-
return getMode(copySpec, "getDirPermissions", () -> copySpec.getDirMode());
151+
private Integer getUnixNumericDirPermissions(CopySpec copySpec) {
152+
return (GradleVersion.current().compareTo(GradleVersion.version("8.3")) >= 0)
153+
? asUnixNumeric(copySpec.getDirPermissions()) : getDirMode(copySpec);
154154
}
155155

156-
private Integer getFileMode(CopySpec copySpec) {
157-
return getMode(copySpec, "getFilePermissions", () -> copySpec.getFileMode());
156+
private Integer getUnixNumericFilePermissions(CopySpec copySpec) {
157+
return (GradleVersion.current().compareTo(GradleVersion.version("8.3")) >= 0)
158+
? asUnixNumeric(copySpec.getFilePermissions()) : getFileMode(copySpec);
158159
}
159160

160-
@SuppressWarnings("unchecked")
161-
private Integer getMode(CopySpec copySpec, String methodName, Supplier<Integer> fallback) {
162-
if (GradleVersion.current().compareTo(GradleVersion.version("8.3")) >= 0) {
163-
try {
164-
Object filePermissions = ((Property<Object>) copySpec.getClass().getMethod(methodName).invoke(copySpec))
165-
.getOrNull();
166-
return (filePermissions != null)
167-
? (int) filePermissions.getClass().getMethod("toUnixNumeric").invoke(filePermissions) : null;
168-
}
169-
catch (Exception ex) {
170-
throw new GradleException("Failed to get permissions", ex);
171-
}
172-
}
173-
return fallback.get();
161+
private Integer asUnixNumeric(Property<ConfigurableFilePermissions> permissions) {
162+
return permissions.isPresent() ? permissions.get().toUnixNumeric() : null;
163+
}
164+
165+
@SuppressWarnings("deprecation")
166+
private Integer getDirMode(CopySpec copySpec) {
167+
return copySpec.getDirMode();
168+
}
169+
170+
@SuppressWarnings("deprecation")
171+
private Integer getFileMode(CopySpec copySpec) {
172+
return copySpec.getFileMode();
174173
}
175174

176175
private boolean isUsingDefaultLoader(Jar jar) {

‎spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/src/main/java/org/springframework/boot/gradle/tasks/bundling/BootZipCopyAction.java

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
import java.io.InputStream;
2323
import java.io.OutputStream;
2424
import java.io.OutputStreamWriter;
25-
import java.lang.reflect.Method;
2625
import java.time.OffsetDateTime;
2726
import java.time.ZoneOffset;
2827
import java.util.Collection;
@@ -488,17 +487,12 @@ private int getFileMode(FileCopyDetails details) {
488487
}
489488

490489
private int getPermissions(FileCopyDetails details) {
491-
if (GradleVersion.current().compareTo(GradleVersion.version("8.3")) >= 0) {
492-
try {
493-
Method getPermissionsMethod = details.getClass().getMethod("getPermissions");
494-
getPermissionsMethod.setAccessible(true);
495-
Object permissions = getPermissionsMethod.invoke(details);
496-
return (int) permissions.getClass().getMethod("toUnixNumeric").invoke(permissions);
497-
}
498-
catch (Exception ex) {
499-
throw new GradleException("Failed to get permissions", ex);
500-
}
501-
}
490+
return (GradleVersion.current().compareTo(GradleVersion.version("8.3")) >= 0)
491+
? details.getPermissions().toUnixNumeric() : getMode(details);
492+
}
493+
494+
@SuppressWarnings("deprecation")
495+
private int getMode(FileCopyDetails details) {
502496
return details.getMode();
503497
}
504498

‎spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/src/test/java/org/springframework/boot/gradle/junit/GradleProjectBuilder.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import org.gradle.api.JavaVersion;
2222
import org.gradle.api.Project;
2323
import org.gradle.internal.nativeintegration.services.NativeServices;
24+
import org.gradle.internal.nativeintegration.services.NativeServices.NativeServicesMode;
2425
import org.gradle.testfixtures.ProjectBuilder;
2526
import org.gradle.testfixtures.internal.ProjectBuilderImpl;
2627

@@ -68,7 +69,7 @@ public Project build() {
6869
builder.withName(this.name);
6970
}
7071
if (JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_17)) {
71-
NativeServices.initializeOnClient(userHome);
72+
NativeServices.initializeOnClient(userHome, NativeServicesMode.ENABLED);
7273
try {
7374
ProjectBuilderImpl.getGlobalServices();
7475
}

‎spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/src/test/java/org/springframework/boot/gradle/plugin/KotlinPluginActionIntegrationTests.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ void taskConfigurationIsAvoided() throws IOException {
8282
configured.add(line.substring("Configuring :".length()));
8383
}
8484
}
85-
assertThat(configured).containsExactlyInAnyOrder("help", "clean");
85+
assertThat(configured).containsExactlyInAnyOrder("help", "compileJava", "clean");
8686
}
8787

8888
}

0 commit comments

Comments
 (0)