KotlinBuildScriptDependenciesResolver should return valid javaHome

[NataliaUkhorskaya]

KotlinBuildScriptDependenciesResolver should contains javaHome in the returned script dependencies. Otherwise gradle.kts scripts can be highlighted with errors if non-java sdk is set as Project SDK in Intellij IDEA

related tiked: https://youtrack.jetbrains.com/issue/KT-21501

Expected Behavior

ScriptDependencies.javaHome is set to some valid value. It can be obtained from environment parameter passed into resolve method.

Current Behavior

javaHome is null

[eskatos]

This will ship in Gradle 5.4

[eskatos]

@NataliaUkhorskaya, this shipped in Gradle 5.4
We are seeing some issues with IntelliJ Kotlin plugin 1.3.30 not recognizing JDK types in the script editor:

Could it be related?

[JLLeitschuh]

@eskatos This looks like a Java 10+ issue. I'm guessing you wouldn't see that problem if the language version was set to Java 8. Just a thought.

[wolfs]

Using Java 8 in some places makes the problem go away...

[eskatos]

The scripts already use Java 8 language version.

But yes, if the build is imported using Java 11 then this problem shows up, it doesn't happen if the build was imported using Java 8.

[eskatos]

In my comment above it worked for me importing with Java 8 because both Gradle
JVM and Project SDK were the same, not demonstrating the problem.

The root problem is that the Kotlin DSL IDEA script editor can't find JDK classes with Gradle 5.4 if Gradle JVM != Project SDK. See gradle/gradle#9195