From 1ccd8dae8ab6b1e9edcd15802cc2ded91fb79147 Mon Sep 17 00:00:00 2001 From: dreis2211 Date: Wed, 10 Feb 2021 15:42:38 +0100 Subject: [PATCH 1/2] Allow the project to be built with Java 16 See gh-25171 --- .../boot/build/JavaConventions.java | 30 ++--- .../build/toolchain/ToolchainExtension.java | 68 ++++++++++ .../boot/build/toolchain/ToolchainPlugin.java | 116 ++++++++++++++++++ settings.gradle | 4 +- .../spring-boot-cli/build.gradle | 4 + .../boot/test/json/JsonbTesterTests.java | 2 +- .../spring-boot-gradle-plugin/build.gradle | 4 + .../spring-boot-loader-tools/build.gradle | 2 +- .../boot/maven/JarIntegrationTests.java | 3 + 9 files changed, 209 insertions(+), 24 deletions(-) create mode 100644 buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainExtension.java create mode 100644 buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainPlugin.java diff --git a/buildSrc/src/main/java/org/springframework/boot/build/JavaConventions.java b/buildSrc/src/main/java/org/springframework/boot/build/JavaConventions.java index 6ef28b12413..6e03dca09d7 100644 --- a/buildSrc/src/main/java/org/springframework/boot/build/JavaConventions.java +++ b/buildSrc/src/main/java/org/springframework/boot/build/JavaConventions.java @@ -16,14 +16,12 @@ package org.springframework.boot.build; -import java.io.File; import java.util.Arrays; import java.util.Collections; import java.util.List; import java.util.Map; import java.util.Set; import java.util.TreeMap; -import java.util.function.Consumer; import java.util.stream.Collectors; import io.spring.javaformat.gradle.FormatTask; @@ -49,6 +47,7 @@ import org.gradle.testretry.TestRetryTaskExtension; import org.springframework.boot.build.optional.OptionalDependenciesPlugin; import org.springframework.boot.build.testing.TestFailuresPlugin; +import org.springframework.boot.build.toolchain.ToolchainPlugin; /** * Conventions that are applied in the presence of the {@link JavaBasePlugin}. When the @@ -103,6 +102,7 @@ class JavaConventions { configureTestConventions(project); configureJarManifestConventions(project); configureDependencyManagement(project); + configureToolchain(project); }); } @@ -145,7 +145,6 @@ class JavaConventions { private void configureTestConventions(Project project) { project.getTasks().withType(Test.class, (test) -> { - withOptionalBuildJavaHome(project, (javaHome) -> test.setExecutable(javaHome + "/bin/java")); test.useJUnitPlatform(); test.setMaxHeapSize("1024M"); }); @@ -165,38 +164,25 @@ class JavaConventions { } private void configureJavadocConventions(Project project) { - project.getTasks().withType(Javadoc.class, (javadoc) -> { - javadoc.getOptions().source("1.8").encoding("UTF-8"); - withOptionalBuildJavaHome(project, (javaHome) -> javadoc.setExecutable(javaHome + "/bin/javadoc")); - }); + project.getTasks().withType(Javadoc.class, (javadoc) -> javadoc.getOptions().source("1.8").encoding("UTF-8")); } private void configureJavaCompileConventions(Project project) { project.getTasks().withType(JavaCompile.class, (compile) -> { compile.getOptions().setEncoding("UTF-8"); - withOptionalBuildJavaHome(project, (javaHome) -> { - compile.getOptions().setFork(true); - compile.getOptions().getForkOptions().setJavaHome(new File(javaHome)); - compile.getOptions().getForkOptions().setExecutable(javaHome + "/bin/javac"); - }); + compile.setSourceCompatibility("1.8"); + compile.setTargetCompatibility("1.8"); List args = compile.getOptions().getCompilerArgs(); if (!args.contains("-parameters")) { args.add("-parameters"); } - if (JavaVersion.current() == JavaVersion.VERSION_1_8) { + if (!project.hasProperty("toolchainVersion") && JavaVersion.current() == JavaVersion.VERSION_1_8) { args.addAll(Arrays.asList("-Werror", "-Xlint:unchecked", "-Xlint:deprecation", "-Xlint:rawtypes", "-Xlint:varargs")); } }); } - private void withOptionalBuildJavaHome(Project project, Consumer consumer) { - String buildJavaHome = (String) project.findProperty("buildJavaHome"); - if (buildJavaHome != null && !buildJavaHome.isEmpty()) { - consumer.accept(buildJavaHome); - } - } - private void configureSpringJavaFormat(Project project) { project.getPlugins().apply(SpringJavaFormatPlugin.class); project.getTasks().withType(FormatTask.class, (formatTask) -> formatTask.setEncoding("UTF-8")); @@ -228,4 +214,8 @@ class JavaConventions { .getByName(OptionalDependenciesPlugin.OPTIONAL_CONFIGURATION_NAME).extendsFrom(dependencyManagement)); } + private void configureToolchain(Project project) { + project.getPlugins().apply(ToolchainPlugin.class); + } + } diff --git a/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainExtension.java b/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainExtension.java new file mode 100644 index 00000000000..eb1a7d1b48d --- /dev/null +++ b/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainExtension.java @@ -0,0 +1,68 @@ +/* + * Copyright 2012-2021 the original author or authors. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * https://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.springframework.boot.build.toolchain; + +import java.util.Optional; + +import org.gradle.api.Project; +import org.gradle.jvm.toolchain.JavaLanguageVersion; + +/** + * DSL extension for {@link ToolchainPlugin}. + * + * @author Christoph Dreis + */ +public class ToolchainExtension { + + private final Project project; + + private int maximumCompatibleJavaVersion; + + public ToolchainExtension(Project project) { + this.project = project; + } + + public void setMaximumCompatibleJavaVersion(int maximumVersion) { + this.maximumCompatibleJavaVersion = maximumVersion; + } + + public Optional getToolchainVersion() { + String toolchainVersion = (String) this.project.findProperty("toolchainVersion"); + if (toolchainVersion == null) { + return Optional.empty(); + } + int version = Integer.parseInt(toolchainVersion); + return getJavaLanguageVersion(version); + } + + public boolean isJavaVersionSupported() { + Optional maximumVersion = getJavaLanguageVersion(this.maximumCompatibleJavaVersion); + if (!maximumVersion.isPresent()) { + return true; + } + Optional toolchainVersion = getToolchainVersion(); + return toolchainVersion.isPresent() && maximumVersion.get().canCompileOrRun(toolchainVersion.get()); + } + + private Optional getJavaLanguageVersion(int version) { + if (version >= 8) { + return Optional.of(JavaLanguageVersion.of(version)); + } + return Optional.empty(); + } + +} diff --git a/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainPlugin.java b/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainPlugin.java new file mode 100644 index 00000000000..aedacfaf114 --- /dev/null +++ b/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainPlugin.java @@ -0,0 +1,116 @@ +/* + * Copyright 2012-2021 the original author or authors. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * https://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.springframework.boot.build.toolchain; + +import java.util.Arrays; +import java.util.List; +import java.util.Optional; + +import org.gradle.api.Action; +import org.gradle.api.Plugin; +import org.gradle.api.Project; +import org.gradle.api.tasks.compile.JavaCompile; +import org.gradle.api.tasks.javadoc.Javadoc; +import org.gradle.api.tasks.testing.Test; +import org.gradle.jvm.toolchain.JavaLanguageVersion; +import org.gradle.jvm.toolchain.JavaToolchainService; +import org.gradle.jvm.toolchain.JavaToolchainSpec; + +/** + * {@link Plugin} for customizing Gradle's toolchain support. + * + * @author Christoph Dreis + */ +public class ToolchainPlugin implements Plugin { + + @Override + public void apply(Project project) { + configureToolchain(project); + } + + private void configureToolchain(Project project) { + ToolchainExtension toolchain = project.getExtensions().create("toolchain", ToolchainExtension.class, project); + project.afterEvaluate((evaluated) -> { + Optional toolchainVersion = toolchain.getToolchainVersion(); + if (toolchainVersion.isPresent()) { + if (!toolchain.isJavaVersionSupported()) { + disableToolchainTasks(project); + } + else { + configureJavaCompileToolchain(project, toolchain); + configureJavadocToolchain(project, toolchain); + configureTestToolchain(project, toolchain); + } + } + }); + } + + private void disableToolchainTasks(Project project) { + project.getTasks().withType(JavaCompile.class, (task) -> task.setEnabled(false)); + project.getTasks().withType(Javadoc.class, (task) -> task.setEnabled(false)); + project.getTasks().withType(Test.class, (task) -> task.setEnabled(false)); + } + + private void configureJavaCompileToolchain(Project project, ToolchainExtension toolchain) { + project.getTasks().withType(JavaCompile.class, (compile) -> { + withOptionalJavaToolchain(toolchain).ifPresent((action) -> { + JavaToolchainService service = getJavaToolchainService(project); + compile.getJavaCompiler().set(service.compilerFor(action)); + compile.getOptions().setFork(true); + // See https://github.com/gradle/gradle/issues/15538 + List forkArgs = Arrays.asList("--add-opens", + "jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED"); + compile.getOptions().getForkOptions().getJvmArgs().addAll(forkArgs); + }); + }); + } + + private void configureJavadocToolchain(Project project, ToolchainExtension toolchain) { + project.getTasks().withType(Javadoc.class, (javadoc) -> { + withOptionalJavaToolchain(toolchain).ifPresent((action) -> { + JavaToolchainService service = getJavaToolchainService(project); + javadoc.getJavadocTool().set(service.javadocToolFor(action)); + }); + }); + } + + private void configureTestToolchain(Project project, ToolchainExtension toolchain) { + project.getTasks().withType(Test.class, (test) -> { + withOptionalJavaToolchain(toolchain).ifPresent((action) -> { + JavaToolchainService service = getJavaToolchainService(project); + test.getJavaLauncher().set(service.launcherFor(action)); + // See https://github.com/spring-projects/spring-ldap/issues/570 + List arguments = Arrays.asList("--add-exports=java.naming/com.sun.jndi.ldap=ALL-UNNAMED", + "--illegal-access=warn"); + test.jvmArgs(arguments); + }); + }); + } + + private JavaToolchainService getJavaToolchainService(Project project) { + return project.getExtensions().getByType(JavaToolchainService.class); + } + + private Optional> withOptionalJavaToolchain(ToolchainExtension toolchain) { + return toolchain.getToolchainVersion().map((toolchainVersion) -> { + Action action = (javaToolchainSpec) -> javaToolchainSpec.getLanguageVersion() + .convention(toolchainVersion); + return Optional.of(action); + }).orElse(Optional.empty()); + } + +} diff --git a/settings.gradle b/settings.gradle index 45b4a5f4f32..301786b0214 100644 --- a/settings.gradle +++ b/settings.gradle @@ -31,8 +31,8 @@ rootProject.name="spring-boot-build" settings.gradle.projectsLoaded { gradleEnterprise { buildScan { - if (settings.gradle.rootProject.hasProperty('buildJavaHome')) { - value('Build Java home', settings.gradle.rootProject.getProperty('buildJavaHome')) + if (settings.gradle.rootProject.hasProperty('toolchainVersion')) { + value('Toolchain Version', settings.gradle.rootProject.getProperty('toolchainVersion')) } settings.gradle.rootProject.getBuildDir().mkdirs() diff --git a/spring-boot-project/spring-boot-cli/build.gradle b/spring-boot-project/spring-boot-cli/build.gradle index 3172eb5e173..f0a3b1f261a 100644 --- a/spring-boot-project/spring-boot-cli/build.gradle +++ b/spring-boot-project/spring-boot-cli/build.gradle @@ -7,6 +7,10 @@ plugins { description = "Spring Boot CLI" +toolchain { + maximumCompatibleJavaVersion = 15 +} + configurations { dependenciesBom loader diff --git a/spring-boot-project/spring-boot-test/src/test/java/org/springframework/boot/test/json/JsonbTesterTests.java b/spring-boot-project/spring-boot-test/src/test/java/org/springframework/boot/test/json/JsonbTesterTests.java index 29892aeb8fe..92237238c2c 100644 --- a/spring-boot-project/spring-boot-test/src/test/java/org/springframework/boot/test/json/JsonbTesterTests.java +++ b/spring-boot-project/spring-boot-test/src/test/java/org/springframework/boot/test/json/JsonbTesterTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2019 the original author or authors. + * Copyright 2012-2021 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. diff --git a/spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle b/spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle index 5907d10d8fa..184dcacbc1e 100644 --- a/spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle +++ b/spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle @@ -10,6 +10,10 @@ plugins { description = "Spring Boot Gradle Plugin" +toolchain { + maximumCompatibleJavaVersion = 15 +} + configurations { asciidoctorExtensions { extendsFrom dependencyManagement diff --git a/spring-boot-project/spring-boot-tools/spring-boot-loader-tools/build.gradle b/spring-boot-project/spring-boot-tools/spring-boot-loader-tools/build.gradle index 2d162f7079c..c31d7d4aae5 100644 --- a/spring-boot-project/spring-boot-tools/spring-boot-loader-tools/build.gradle +++ b/spring-boot-project/spring-boot-tools/spring-boot-loader-tools/build.gradle @@ -69,7 +69,7 @@ processResources { } compileJava { - if ((!project.hasProperty("buildJavaHome")) && JavaVersion.current() == JavaVersion.VERSION_1_8) { + if ((!project.hasProperty("toolchainVersion")) && JavaVersion.current() == JavaVersion.VERSION_1_8) { options.compilerArgs += ['-Xlint:-sunapi', '-XDenableSunApiLintControl'] } } diff --git a/spring-boot-project/spring-boot-tools/spring-boot-maven-plugin/src/intTest/java/org/springframework/boot/maven/JarIntegrationTests.java b/spring-boot-project/spring-boot-tools/spring-boot-maven-plugin/src/intTest/java/org/springframework/boot/maven/JarIntegrationTests.java index 0d60105a7fb..0a9edc6b338 100644 --- a/spring-boot-project/spring-boot-tools/spring-boot-maven-plugin/src/intTest/java/org/springframework/boot/maven/JarIntegrationTests.java +++ b/spring-boot-project/spring-boot-tools/spring-boot-maven-plugin/src/intTest/java/org/springframework/boot/maven/JarIntegrationTests.java @@ -30,6 +30,8 @@ import java.util.stream.Collectors; import java.util.zip.ZipEntry; import org.junit.jupiter.api.TestTemplate; +import org.junit.jupiter.api.condition.DisabledForJreRange; +import org.junit.jupiter.api.condition.JRE; import org.junit.jupiter.api.extension.ExtendWith; import org.springframework.boot.loader.tools.FileUtils; @@ -247,6 +249,7 @@ class JarIntegrationTests extends AbstractArchiveIntegrationTests { }); } + @DisabledForJreRange(min = JRE.JAVA_16) // Remove this once Kotlin supports Java 16 @TestTemplate void whenAProjectUsesKotlinItsModuleMetadataIsRepackagedIntoBootInfClasses(MavenBuild mavenBuild) { mavenBuild.project("jar-with-kotlin-module").execute((project) -> { From 1ce6e796feb787f3fe9b583733228ee5b4524655 Mon Sep 17 00:00:00 2001 From: Andy Wilkinson Date: Thu, 4 Mar 2021 18:54:18 +0000 Subject: [PATCH 2/2] Polish "Allow the project to be built with Java 16" See gh-25171 --- .../build/toolchain/ToolchainExtension.java | 40 +++------ .../boot/build/toolchain/ToolchainPlugin.java | 85 +++++++------------ .../spring-boot-cli/build.gradle | 2 +- .../spring-boot-gradle-plugin/build.gradle | 2 +- 4 files changed, 43 insertions(+), 86 deletions(-) diff --git a/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainExtension.java b/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainExtension.java index eb1a7d1b48d..efa3731a435 100644 --- a/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainExtension.java +++ b/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainExtension.java @@ -16,9 +16,8 @@ package org.springframework.boot.build.toolchain; -import java.util.Optional; - import org.gradle.api.Project; +import org.gradle.api.provider.Property; import org.gradle.jvm.toolchain.JavaLanguageVersion; /** @@ -28,41 +27,22 @@ import org.gradle.jvm.toolchain.JavaLanguageVersion; */ public class ToolchainExtension { - private final Project project; + private final Property maximumCompatibleJavaVersion; - private int maximumCompatibleJavaVersion; + private final JavaLanguageVersion javaVersion; public ToolchainExtension(Project project) { - this.project = project; + this.maximumCompatibleJavaVersion = project.getObjects().property(JavaLanguageVersion.class); + String toolchainVersion = (String) project.findProperty("toolchainVersion"); + this.javaVersion = (toolchainVersion != null) ? JavaLanguageVersion.of(toolchainVersion) : null; } - public void setMaximumCompatibleJavaVersion(int maximumVersion) { - this.maximumCompatibleJavaVersion = maximumVersion; + public Property getMaximumCompatibleJavaVersion() { + return this.maximumCompatibleJavaVersion; } - public Optional getToolchainVersion() { - String toolchainVersion = (String) this.project.findProperty("toolchainVersion"); - if (toolchainVersion == null) { - return Optional.empty(); - } - int version = Integer.parseInt(toolchainVersion); - return getJavaLanguageVersion(version); - } - - public boolean isJavaVersionSupported() { - Optional maximumVersion = getJavaLanguageVersion(this.maximumCompatibleJavaVersion); - if (!maximumVersion.isPresent()) { - return true; - } - Optional toolchainVersion = getToolchainVersion(); - return toolchainVersion.isPresent() && maximumVersion.get().canCompileOrRun(toolchainVersion.get()); - } - - private Optional getJavaLanguageVersion(int version) { - if (version >= 8) { - return Optional.of(JavaLanguageVersion.of(version)); - } - return Optional.empty(); + JavaLanguageVersion getJavaVersion() { + return this.javaVersion; } } diff --git a/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainPlugin.java b/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainPlugin.java index aedacfaf114..093d5f3c65c 100644 --- a/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainPlugin.java +++ b/buildSrc/src/main/java/org/springframework/boot/build/toolchain/ToolchainPlugin.java @@ -18,16 +18,14 @@ package org.springframework.boot.build.toolchain; import java.util.Arrays; import java.util.List; -import java.util.Optional; -import org.gradle.api.Action; import org.gradle.api.Plugin; import org.gradle.api.Project; +import org.gradle.api.plugins.JavaPluginExtension; import org.gradle.api.tasks.compile.JavaCompile; import org.gradle.api.tasks.javadoc.Javadoc; import org.gradle.api.tasks.testing.Test; import org.gradle.jvm.toolchain.JavaLanguageVersion; -import org.gradle.jvm.toolchain.JavaToolchainService; import org.gradle.jvm.toolchain.JavaToolchainSpec; /** @@ -44,19 +42,28 @@ public class ToolchainPlugin implements Plugin { private void configureToolchain(Project project) { ToolchainExtension toolchain = project.getExtensions().create("toolchain", ToolchainExtension.class, project); - project.afterEvaluate((evaluated) -> { - Optional toolchainVersion = toolchain.getToolchainVersion(); - if (toolchainVersion.isPresent()) { - if (!toolchain.isJavaVersionSupported()) { - disableToolchainTasks(project); - } - else { - configureJavaCompileToolchain(project, toolchain); - configureJavadocToolchain(project, toolchain); - configureTestToolchain(project, toolchain); - } - } - }); + JavaLanguageVersion toolchainVersion = toolchain.getJavaVersion(); + if (toolchainVersion != null) { + project.afterEvaluate((evaluated) -> configure(evaluated, toolchain)); + } + } + + private void configure(Project project, ToolchainExtension toolchain) { + if (!isJavaVersionSupported(toolchain, toolchain.getJavaVersion())) { + disableToolchainTasks(project); + } + else { + JavaToolchainSpec toolchainSpec = project.getExtensions().getByType(JavaPluginExtension.class) + .getToolchain(); + toolchainSpec.getLanguageVersion().set(toolchain.getJavaVersion()); + configureJavaCompileToolchain(project, toolchain); + configureTestToolchain(project, toolchain); + } + } + + public boolean isJavaVersionSupported(ToolchainExtension toolchain, JavaLanguageVersion toolchainVersion) { + return toolchain.getMaximumCompatibleJavaVersion().map((version) -> version.canCompileOrRun(toolchainVersion)) + .getOrElse(true); } private void disableToolchainTasks(Project project) { @@ -67,50 +74,20 @@ public class ToolchainPlugin implements Plugin { private void configureJavaCompileToolchain(Project project, ToolchainExtension toolchain) { project.getTasks().withType(JavaCompile.class, (compile) -> { - withOptionalJavaToolchain(toolchain).ifPresent((action) -> { - JavaToolchainService service = getJavaToolchainService(project); - compile.getJavaCompiler().set(service.compilerFor(action)); - compile.getOptions().setFork(true); - // See https://github.com/gradle/gradle/issues/15538 - List forkArgs = Arrays.asList("--add-opens", - "jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED"); - compile.getOptions().getForkOptions().getJvmArgs().addAll(forkArgs); - }); - }); - } - - private void configureJavadocToolchain(Project project, ToolchainExtension toolchain) { - project.getTasks().withType(Javadoc.class, (javadoc) -> { - withOptionalJavaToolchain(toolchain).ifPresent((action) -> { - JavaToolchainService service = getJavaToolchainService(project); - javadoc.getJavadocTool().set(service.javadocToolFor(action)); - }); + compile.getOptions().setFork(true); + // See https://github.com/gradle/gradle/issues/15538 + List forkArgs = Arrays.asList("--add-opens", "jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED"); + compile.getOptions().getForkOptions().getJvmArgs().addAll(forkArgs); }); } private void configureTestToolchain(Project project, ToolchainExtension toolchain) { project.getTasks().withType(Test.class, (test) -> { - withOptionalJavaToolchain(toolchain).ifPresent((action) -> { - JavaToolchainService service = getJavaToolchainService(project); - test.getJavaLauncher().set(service.launcherFor(action)); - // See https://github.com/spring-projects/spring-ldap/issues/570 - List arguments = Arrays.asList("--add-exports=java.naming/com.sun.jndi.ldap=ALL-UNNAMED", - "--illegal-access=warn"); - test.jvmArgs(arguments); - }); + // See https://github.com/spring-projects/spring-ldap/issues/570 + List arguments = Arrays.asList("--add-exports=java.naming/com.sun.jndi.ldap=ALL-UNNAMED", + "--illegal-access=warn"); + test.jvmArgs(arguments); }); } - private JavaToolchainService getJavaToolchainService(Project project) { - return project.getExtensions().getByType(JavaToolchainService.class); - } - - private Optional> withOptionalJavaToolchain(ToolchainExtension toolchain) { - return toolchain.getToolchainVersion().map((toolchainVersion) -> { - Action action = (javaToolchainSpec) -> javaToolchainSpec.getLanguageVersion() - .convention(toolchainVersion); - return Optional.of(action); - }).orElse(Optional.empty()); - } - } diff --git a/spring-boot-project/spring-boot-cli/build.gradle b/spring-boot-project/spring-boot-cli/build.gradle index f0a3b1f261a..e15ff8b2388 100644 --- a/spring-boot-project/spring-boot-cli/build.gradle +++ b/spring-boot-project/spring-boot-cli/build.gradle @@ -8,7 +8,7 @@ plugins { description = "Spring Boot CLI" toolchain { - maximumCompatibleJavaVersion = 15 + maximumCompatibleJavaVersion = JavaLanguageVersion.of(15) } configurations { diff --git a/spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle b/spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle index 184dcacbc1e..eaa866b8d2a 100644 --- a/spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle +++ b/spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle @@ -11,7 +11,7 @@ plugins { description = "Spring Boot Gradle Plugin" toolchain { - maximumCompatibleJavaVersion = 15 + maximumCompatibleJavaVersion = JavaLanguageVersion.of(15) } configurations {