From f663b8f554978b258ad94fae5d2cb83e4de1fb73 Mon Sep 17 00:00:00 2001 From: Alexander Udalov Date: Fri, 10 Aug 2018 18:34:40 +0200 Subject: [PATCH] Fix kotlin-maven-plugin compilation --- .../cli/jvm/compiler/KotlinToJVMBytecodeCompiler.kt | 3 ++- .../jetbrains/kotlin/maven/ExecuteKotlinScriptMojo.java | 8 +++----- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/compiler/cli/src/org/jetbrains/kotlin/cli/jvm/compiler/KotlinToJVMBytecodeCompiler.kt b/compiler/cli/src/org/jetbrains/kotlin/cli/jvm/compiler/KotlinToJVMBytecodeCompiler.kt index 7a1d7933b3d..547058cd2e1 100644 --- a/compiler/cli/src/org/jetbrains/kotlin/cli/jvm/compiler/KotlinToJVMBytecodeCompiler.kt +++ b/compiler/cli/src/org/jetbrains/kotlin/cli/jvm/compiler/KotlinToJVMBytecodeCompiler.kt @@ -346,7 +346,8 @@ object KotlinToJVMBytecodeCompiler { } } - private fun analyzeAndGenerate(environment: KotlinCoreEnvironment): GenerationState? { + @Suppress("MemberVisibilityCanBePrivate") // Used in ExecuteKotlinScriptMojo + fun analyzeAndGenerate(environment: KotlinCoreEnvironment): GenerationState? { val result = repeatAnalysisIfNeeded(analyze(environment, null), environment, null) ?: return null if (!result.shouldGenerateCode) return null diff --git a/libraries/tools/kotlin-maven-plugin/src/main/java/org/jetbrains/kotlin/maven/ExecuteKotlinScriptMojo.java b/libraries/tools/kotlin-maven-plugin/src/main/java/org/jetbrains/kotlin/maven/ExecuteKotlinScriptMojo.java index a82a890ed2a..504e8be96e7 100644 --- a/libraries/tools/kotlin-maven-plugin/src/main/java/org/jetbrains/kotlin/maven/ExecuteKotlinScriptMojo.java +++ b/libraries/tools/kotlin-maven-plugin/src/main/java/org/jetbrains/kotlin/maven/ExecuteKotlinScriptMojo.java @@ -34,6 +34,7 @@ import org.apache.maven.plugins.annotations.ResolutionScope; import org.apache.maven.project.MavenProject; import org.codehaus.plexus.component.repository.ComponentDependency; import org.jetbrains.kotlin.cli.common.CLIConfigurationKeys; +import org.jetbrains.kotlin.cli.common.config.KotlinSourceRoot; import org.jetbrains.kotlin.cli.jvm.K2JVMCompiler; import org.jetbrains.kotlin.cli.jvm.compiler.EnvironmentConfigFiles; import org.jetbrains.kotlin.cli.jvm.compiler.KotlinCoreEnvironment; @@ -43,14 +44,11 @@ import org.jetbrains.kotlin.codegen.GeneratedClassLoader; import org.jetbrains.kotlin.codegen.state.GenerationState; import org.jetbrains.kotlin.config.CommonConfigurationKeys; import org.jetbrains.kotlin.config.CompilerConfiguration; -import org.jetbrains.kotlin.config.JVMConfigurationKeys; -import org.jetbrains.kotlin.config.KotlinSourceRoot; import org.jetbrains.kotlin.load.java.JvmAbi; import org.jetbrains.kotlin.name.FqName; import org.jetbrains.kotlin.psi.KtScript; import org.jetbrains.kotlin.script.ReflectionUtilKt; import org.jetbrains.kotlin.scripting.compiler.plugin.ScriptingCompilerConfigurationExtensionKt; -import org.jetbrains.kotlin.utils.PathUtil; import java.io.File; import java.io.FileOutputStream; @@ -174,14 +172,14 @@ public class ExecuteKotlinScriptMojo extends AbstractMojo { for (File item: deps) { if (item.exists()) { - configuration.add(JVMConfigurationKeys.CONTENT_ROOTS, new JvmClasspathRoot(item)); + configuration.add(CLIConfigurationKeys.CONTENT_ROOTS, new JvmClasspathRoot(item)); getLog().debug("Adding to classpath: " + item.getAbsolutePath()); } else { getLog().debug("Skipping non-existing dependency: " + item.getAbsolutePath()); } } - configuration.add(JVMConfigurationKeys.CONTENT_ROOTS, new KotlinSourceRoot(scriptFile.getAbsolutePath())); + configuration.add(CLIConfigurationKeys.CONTENT_ROOTS, new KotlinSourceRoot(scriptFile.getAbsolutePath())); configuration.put(CommonConfigurationKeys.MODULE_NAME, JvmAbi.DEFAULT_MODULE_NAME); ScriptingCompilerConfigurationExtensionKt.configureScriptDefinitions(