Skip to content

Commit

Permalink
Fix GraalVM integration test on GitHub Actions (#3626)
Browse files Browse the repository at this point in the history
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)
  • Loading branch information
marcphilipp committed Jun 17, 2024
1 parent 43ddd64 commit 4bda14d
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ org.gradle.jvmargs=-Xmx1g -XX:MaxMetaspaceSize=512m -XX:+HeapDumpOnOutOfMemoryEr
--add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
org.gradle.caching=true
org.gradle.parallel=true
org.gradle.java.installations.fromEnv=JDK8,JDK18,JDK19,JDK20,JDK21,JDK22,GRAALVM_HOME
org.gradle.java.installations.fromEnv=JDK8,JDK18,JDK19,JDK20,JDK21,JDK22
org.gradle.kotlin.dsl.allWarningsAsErrors=true

# Test Distribution
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ void runsTestsInNativeImage() {
.setProject("graalvm-starter") //
.addArguments("-Dmaven.repo=" + MavenRepo.dir()) //
.addArguments("javaToolchains", "nativeTest", "--no-daemon", "--stacktrace") //
.addArguments("-Porg.gradle.java.installations.fromEnv=GRAALVM_HOME") //
.setTimeout(Duration.ofMinutes(5)) //
.build();

Expand Down

0 comments on commit 4bda14d

Please sign in to comment.