Skip to content

Commit 4bda14d

Browse files
committed
Fix GraalVM integration test on GitHub Actions (#3626)
Since gradle-build-action currently writes `~/.gradle/gradle.properties` which takes precedence over the project-local `gradle.properties`, the test now passes `org.gradle.java.installations.fromEnv` as a CLI arg. (cherry picked from commit 89ae123)
1 parent 43ddd64 commit 4bda14d

File tree

3 files changed

+2
-2
lines changed

3 files changed

+2
-2
lines changed

gradle.properties

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ org.gradle.jvmargs=-Xmx1g -XX:MaxMetaspaceSize=512m -XX:+HeapDumpOnOutOfMemoryEr
1919
--add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
2020
org.gradle.caching=true
2121
org.gradle.parallel=true
22-
org.gradle.java.installations.fromEnv=JDK8,JDK18,JDK19,JDK20,JDK21,JDK22,GRAALVM_HOME
22+
org.gradle.java.installations.fromEnv=JDK8,JDK18,JDK19,JDK20,JDK21,JDK22
2323
org.gradle.kotlin.dsl.allWarningsAsErrors=true
2424

2525
# Test Distribution

platform-tooling-support-tests/projects/graalvm-starter/gradle.properties

Lines changed: 0 additions & 1 deletion
This file was deleted.

platform-tooling-support-tests/src/test/java/platform/tooling/support/tests/GraalVmStarterTests.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ void runsTestsInNativeImage() {
4141
.setProject("graalvm-starter") //
4242
.addArguments("-Dmaven.repo=" + MavenRepo.dir()) //
4343
.addArguments("javaToolchains", "nativeTest", "--no-daemon", "--stacktrace") //
44+
.addArguments("-Porg.gradle.java.installations.fromEnv=GRAALVM_HOME") //
4445
.setTimeout(Duration.ofMinutes(5)) //
4546
.build();
4647

0 commit comments

Comments
 (0)