diff --git a/CHANGELOG.md b/CHANGELOG.md index 0c17a950d..58a2578df 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,8 +7,7 @@ * Bug fix to handle export dependencies file path correctly. ### Version 0.53.3 -* Add configuration cleanCacheDir to conditionally delete the cache directory or - just the existing dependency rules files +* Add configuration cleanCacheDir to conditionally delete the cache directory or just the existing dependency rules files ### Version 0.54.0 * Stop using /tmp for Android Lint gen-rule and use buck-out tmp instead @@ -40,3 +39,4 @@ - Updated `actions/setup-java` to v4 with temurin distribution - Removed rxPermissions and XLog dependencies - Updated to Python 3.8 +* Added support for Kotlin 2.2.0 diff --git a/buildSrc/src/main/java/com/uber/okbuck/core/model/jvm/JvmTarget.java b/buildSrc/src/main/java/com/uber/okbuck/core/model/jvm/JvmTarget.java index 626278f2e..f5a482e2d 100644 --- a/buildSrc/src/main/java/com/uber/okbuck/core/model/jvm/JvmTarget.java +++ b/buildSrc/src/main/java/com/uber/okbuck/core/model/jvm/JvmTarget.java @@ -56,12 +56,10 @@ import org.gradle.api.tasks.compile.JavaCompile; import org.gradle.api.tasks.testing.Test; import org.gradle.jvm.tasks.Jar; -import org.jetbrains.kotlin.allopen.gradle.AllOpenGradleSubplugin; import org.jetbrains.kotlin.gradle.dsl.KotlinJvmOptions; import org.jetbrains.kotlin.gradle.dsl.KotlinSingleTargetExtension; import org.jetbrains.kotlin.gradle.plugin.KotlinBasePluginWrapper; import org.jetbrains.kotlin.gradle.plugin.KotlinCompilation; -import org.jetbrains.kotlin.gradle.plugin.SubpluginOption; import org.jetbrains.kotlin.gradle.plugin.mpp.KotlinCommonCompilation; import org.jetbrains.kotlin.gradle.tasks.KotlinCompile; @@ -470,13 +468,7 @@ public Map> getKotlinFriendPaths(boolean isTest) { } protected List getKotlinCompilerPlugins() { - List subpluginOptions = getAllOpenSubpluginOptions(); - - if (subpluginOptions.size() > 0) { - return ImmutableList.of(KotlinManager.KOTLIN_AO_PLUGIN_TARGET); - } else { - return ImmutableList.of(); - } + return ImmutableList.of(); } protected List getKotlinCompilerOptions() { @@ -484,16 +476,7 @@ protected List getKotlinCompilerOptions() { return ImmutableList.of(); } - ImmutableList.Builder optionBuilder = ImmutableList.builder(); - optionBuilder.addAll(readKotlinCompilerArguments()); - - for (SubpluginOption option : getAllOpenSubpluginOptions()) { - optionBuilder.add("-P"); - optionBuilder.add( - "plugin:org.jetbrains.kotlin.allopen:" + option.getKey() + "=" + option.getValue()); - } - - return optionBuilder.build(); + return ImmutableList.copyOf(readKotlinCompilerArguments()); } private List readKotlinCompilerArguments() { @@ -571,27 +554,6 @@ private List readKotlinCompilerArguments() { } } - private ImmutableList getAllOpenSubpluginOptions() { - if (!getProject().getPlugins().hasPlugin(KotlinManager.KOTLIN_ALLOPEN_MODULE)) { - return ImmutableList.of(); - } - - KotlinSingleTargetExtension extension = - getProject().getExtensions().findByType(KotlinSingleTargetExtension.class); - if (extension == null) { - return ImmutableList.of(); - } - - AllOpenGradleSubplugin subPlugin = - (AllOpenGradleSubplugin) - getProject().getPlugins().getPlugin(KotlinManager.KOTLIN_ALLOPEN_MODULE); - KotlinCompilation fakeCompilation = - new KotlinCommonCompilation(extension.getTarget(), "fakeCompilation"); - return new ImmutableList.Builder() - .addAll(subPlugin.applyToCompilation(fakeCompilation).get()) - .build(); - } - @SuppressWarnings("NoFunctionalReturnType") private static Predicate distinctByKey(Function keyExtractor) { Set seen = ConcurrentHashMap.newKeySet(); diff --git a/libraries/kotlinlibrary/src/test/java/demo/AllOpenTest.kt b/libraries/kotlinlibrary/src/test/java/demo/AllOpenTest.kt deleted file mode 100644 index 08369c769..000000000 --- a/libraries/kotlinlibrary/src/test/java/demo/AllOpenTest.kt +++ /dev/null @@ -1,11 +0,0 @@ -package demo - -import org.junit.Test -import org.mockito.Mockito - -class AllOpenTest { - @Test fun f() { - Mockito.mock(Closed::class.java) - Mockito.mock(Closed2::class.java) - } -}