From 649287b689f52a117cc7bdf7725817d4575e441f Mon Sep 17 00:00:00 2001
From: Dmitry Jemerov <yole@jetbrains.com>
Date: Fri, 20 Mar 2015 16:07:21 +0100
Subject: [PATCH] remove KDoc support in Kotlin Gradle plugin

---
 .../tools/kotlin-gradle-plugin-core/pom.xml   |  5 --
 .../jetbrains/kotlin/gradle/tasks/Tasks.kt    | 63 -------------------
 libraries/tools/kotlin-gradle-plugin/pom.xml  |  6 --
 .../kotlin/gradle/plugin/KotlinPlugin.kt      | 18 ------
 .../kotlin/gradle/tasks/TasksProvider.kt      |  7 ---
 .../gradle/BasicKotlinGradlePluginIT.kt       |  8 ---
 .../testProject/kdocProject/build.gradle      | 40 ------------
 .../com/google/common/base/annotations.xml    | 11 ----
 .../kdocProject/src/main/kotlin/helloWorld.kt | 15 -----
 9 files changed, 173 deletions(-)
 delete mode 100644 libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/build.gradle
 delete mode 100644 libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/externalAnnotations/com/google/common/base/annotations.xml
 delete mode 100644 libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/src/main/kotlin/helloWorld.kt

diff --git a/libraries/tools/kotlin-gradle-plugin-core/pom.xml b/libraries/tools/kotlin-gradle-plugin-core/pom.xml
index feab21cc14c..8636d5d7c2a 100644
--- a/libraries/tools/kotlin-gradle-plugin-core/pom.xml
+++ b/libraries/tools/kotlin-gradle-plugin-core/pom.xml
@@ -52,11 +52,6 @@
             <artifactId>kotlin-jdk-annotations</artifactId>
             <version>${project.version}</version>
         </dependency>
-        <dependency>
-            <groupId>org.jetbrains.kotlin</groupId>
-            <artifactId>kdoc</artifactId>
-            <version>${project.version}</version>
-        </dependency>
         <dependency>
             <groupId>org.jetbrains.kotlin</groupId>
             <artifactId>kotlin-stdlib</artifactId>
diff --git a/libraries/tools/kotlin-gradle-plugin-core/src/main/kotlin/org/jetbrains/kotlin/gradle/tasks/Tasks.kt b/libraries/tools/kotlin-gradle-plugin-core/src/main/kotlin/org/jetbrains/kotlin/gradle/tasks/Tasks.kt
index 7808697de49..f4216a08c85 100644
--- a/libraries/tools/kotlin-gradle-plugin-core/src/main/kotlin/org/jetbrains/kotlin/gradle/tasks/Tasks.kt
+++ b/libraries/tools/kotlin-gradle-plugin-core/src/main/kotlin/org/jetbrains/kotlin/gradle/tasks/Tasks.kt
@@ -284,69 +284,6 @@ public open class Kotlin2JsCompile() : AbstractKotlinCompile<K2JSCompilerArgumen
     }
 }
 
-public open class KDoc() : SourceTask() {
-
-    private val logger = Logging.getLogger(this.javaClass)
-    override fun getLogger() = logger
-
-    public var kdocArgs: KDocArguments = KDocArguments()
-
-    public var destinationDir: File? = null
-
-    init {
-        // by default, output dir is not defined in options
-        kdocArgs.docConfig.docOutputDir = ""
-    }
-
-    TaskAction fun generateDocs() {
-        val args = KDocArguments()
-        val cfg = args.docConfig
-
-        val kdocOptions = kdocArgs.docConfig
-
-        cfg.docOutputDir = if ((kdocOptions.docOutputDir.length() == 0) && (destinationDir != null)) {
-            destinationDir!!.path
-        } else {
-            kdocOptions.docOutputDir
-        }
-        cfg.title = kdocOptions.title
-        cfg.sourceRootHref = kdocOptions.sourceRootHref
-        cfg.projectRootDir = kdocOptions.projectRootDir
-        cfg.warnNoComments = kdocOptions.warnNoComments
-
-        cfg.packagePrefixToUrls.putAll(kdocOptions.packagePrefixToUrls)
-        cfg.ignorePackages.addAll(kdocOptions.ignorePackages)
-        cfg.packageDescriptionFiles.putAll(kdocOptions.packageDescriptionFiles)
-        cfg.packageSummaryText.putAll(kdocOptions.packageSummaryText)
-
-        // KDoc compiler does not accept list of files as input. Try to pass directories instead.
-        args.freeArgs = getSource().map { it.getParentFile()!!.getAbsolutePath() }
-        // Drop compiled sources to temp. Why KDoc compiles anything after all?!
-        args.destination = getTemporaryDir()?.getAbsolutePath()
-
-        getLogger().warn(args.freeArgs.toString())
-        val embeddedAnnotations = getAnnotations(getProject(), getLogger())
-        val userAnnotations = (kdocArgs.annotations ?: "").split(File.pathSeparatorChar).toList()
-        val allAnnotations = if (kdocArgs.noJdkAnnotations) userAnnotations else userAnnotations.plus(embeddedAnnotations.map { it.getPath() })
-        args.annotations = allAnnotations.joinToString(File.pathSeparator)
-
-        args.noStdlib = true
-        args.noJdkAnnotations = true
-
-
-        val compiler = KDocCompiler()
-
-        val messageCollector = GradleMessageCollector(getLogger())
-        val exitCode = compiler.exec(messageCollector, Services.EMPTY, args)
-
-        when (exitCode) {
-            ExitCode.COMPILATION_ERROR -> throw GradleException("Failed to generate kdoc. See log for more details")
-            ExitCode.INTERNAL_ERROR -> throw GradleException("Internal generation error. See log for more details")
-        }
-
-    }
-}
-
 private fun <T: Any> ExtraPropertiesExtension.getOrNull(id: String): T? {
     try {
         @suppress("UNCHECKED_CAST")
diff --git a/libraries/tools/kotlin-gradle-plugin/pom.xml b/libraries/tools/kotlin-gradle-plugin/pom.xml
index 35b74e84c00..9252c989dd8 100644
--- a/libraries/tools/kotlin-gradle-plugin/pom.xml
+++ b/libraries/tools/kotlin-gradle-plugin/pom.xml
@@ -37,12 +37,6 @@
           <version>2.2</version>
           <scope>provided</scope>
         </dependency>
-        <dependency>
-            <groupId>org.jetbrains.kotlin</groupId>
-            <artifactId>kdoc</artifactId>
-            <version>${project.version}</version>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>org.jetbrains.kotlin</groupId>
             <artifactId>kotlin-gradle-plugin-core</artifactId>
diff --git a/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinPlugin.kt b/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinPlugin.kt
index bc462afbd93..5d3f5da4ba1 100644
--- a/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinPlugin.kt
+++ b/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/plugin/KotlinPlugin.kt
@@ -247,7 +247,6 @@ abstract class AbstractKotlinPlugin @Inject constructor(val scriptHandler: Scrip
         project.getPlugins().apply(javaClass<JavaPlugin>())
 
         configureSourceSetDefaults(project as ProjectInternal, javaBasePlugin, javaPluginConvention)
-        configureKDoc(project, javaPluginConvention)
 
         val gradleUtils = GradleUtils(scriptHandler, project)
         project.getExtensions().add(DEFAULT_ANNOTATIONS, gradleUtils.resolveKotlinPluginDependency("kotlin-jdk-annotations"))
@@ -262,23 +261,6 @@ abstract class AbstractKotlinPlugin @Inject constructor(val scriptHandler: Scrip
             }
         })
     }
-
-    open protected fun configureKDoc(project: Project, javaPluginConvention: JavaPluginConvention) {
-        val mainSourceSet = javaPluginConvention.getSourceSets()?.findByName(SourceSet.MAIN_SOURCE_SET_NAME) as HasConvention?
-
-        if (mainSourceSet != null) {
-
-            val kdoc = tasksProvider.createKDocTask(project, KDOC_TASK_NAME)
-
-            kdoc.setDescription("Generates KDoc API documentation for the main source code.")
-            kdoc.setGroup(JavaBasePlugin.DOCUMENTATION_GROUP)
-            kdoc.setSource(mainSourceSet.getConvention().getExtensionsAsDynamicObject().getProperty("kotlin"))
-        }
-
-        project.getTasks().withType(tasksProvider.kDocTaskClass) { it!!.setProperty("destinationDir", File(javaPluginConvention.getDocsDir(), "kdoc")) }
-    }
-
-    public val KDOC_TASK_NAME: String = "kdoc"
 }
 
 
diff --git a/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/tasks/TasksProvider.kt b/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/tasks/TasksProvider.kt
index 26aeff3dbc1..a739f60ec0c 100644
--- a/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/tasks/TasksProvider.kt
+++ b/libraries/tools/kotlin-gradle-plugin/src/main/kotlin/org/jetbrains/kotlin/gradle/tasks/TasksProvider.kt
@@ -19,9 +19,6 @@ public open class KotlinTasksProvider(val tasksLoader: ClassLoader) {
     val kotlinJSCompileTaskClass: Class<AbstractCompile> =
             tasksLoader.loadClass("org.jetbrains.kotlin.gradle.tasks.Kotlin2JsCompile") as Class<AbstractCompile>
 
-    val kDocTaskClass: Class<SourceTask> =
-            tasksLoader.loadClass("org.jetbrains.kotlin.gradle.tasks.KDoc") as Class<SourceTask>
-
     val kotlinJVMOptionsClass: Class<Any> =
             tasksLoader.loadClass("org.jetbrains.kotlin.cli.common.arguments.K2JVMCompilerArguments") as Class<Any>
 
@@ -33,8 +30,4 @@ public open class KotlinTasksProvider(val tasksLoader: ClassLoader) {
     public fun createKotlinJSTask(project: Project, name: String): AbstractCompile {
         return project.getTasks().create(name, kotlinJSCompileTaskClass)
     }
-
-    public fun createKDocTask(project: Project, name: String): SourceTask {
-        return project.getTasks().create(name, kDocTaskClass)
-    }
 }
diff --git a/libraries/tools/kotlin-gradle-plugin/src/test/kotlin/org/jetbrains/kotlin/gradle/BasicKotlinGradlePluginIT.kt b/libraries/tools/kotlin-gradle-plugin/src/test/kotlin/org/jetbrains/kotlin/gradle/BasicKotlinGradlePluginIT.kt
index 02855e22d77..f4eba9f4a71 100644
--- a/libraries/tools/kotlin-gradle-plugin/src/test/kotlin/org/jetbrains/kotlin/gradle/BasicKotlinGradlePluginIT.kt
+++ b/libraries/tools/kotlin-gradle-plugin/src/test/kotlin/org/jetbrains/kotlin/gradle/BasicKotlinGradlePluginIT.kt
@@ -66,14 +66,6 @@ class SimpleKotlinGradleIT : BaseGradleIT() {
         }
     }
 
-    Test fun testSimpleKDoc() {
-        Project("kdocProject", "1.6").build("kdoc") {
-            assertSuccessful()
-            assertReportExists("build/docs/kdoc/demo/MyClass.html")
-            assertContains(":kdoc", "Generating kdoc to")
-        }
-    }
-
     Test fun testKotlinExtraJavaSrc() {
         Project("additionalJavaSrc", "1.6").build("build") {
             assertSuccessful()
diff --git a/libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/build.gradle b/libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/build.gradle
deleted file mode 100644
index a5193474c83..00000000000
--- a/libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/build.gradle
+++ /dev/null
@@ -1,40 +0,0 @@
-buildscript {
-  repositories {
-    mavenCentral()
-    maven {
-        url 'file://' + pathToKotlinPlugin
-    }
-  }
-  dependencies {
-    classpath 'org.jetbrains.kotlin:kotlin-gradle-plugin:0.1-SNAPSHOT'
-  }
-}
-
-
-
-apply plugin: "kotlin"
-
-repositories {
-  maven {
-     url 'file://' + pathToKotlinPlugin
-  }
-  mavenCentral()
-}
-
-dependencies {
-    testCompile  'org.testng:testng:6.8'
-    compile  'org.jetbrains.kotlin:kotlin-stdlib:0.1-SNAPSHOT'
-}
-
-test {
-    useTestNG()
-}
-
-compileKotlin {
-    kotlinOptions.annotations = "externalAnnotations"
-}
-
-
-task wrapper(type: Wrapper) {
-  gradleVersion="1.4"
-}
\ No newline at end of file
diff --git a/libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/externalAnnotations/com/google/common/base/annotations.xml b/libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/externalAnnotations/com/google/common/base/annotations.xml
deleted file mode 100644
index fd541a7e6e5..00000000000
--- a/libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/externalAnnotations/com/google/common/base/annotations.xml
+++ /dev/null
@@ -1,11 +0,0 @@
-<root>
-    <item name='com.google.common.base.Joiner com.google.common.base.Joiner on(java.lang.String)'>
-        <annotation name="org.jetbrains.annotations.NotNull" />
-    </item>
-    <item name='com.google.common.base.Joiner com.google.common.base.Joiner.MapJoiner withKeyValueSeparator(java.lang.String)'>
-        <annotation name="org.jetbrains.annotations.NotNull" />
-    </item>
-    <item name='com.google.common.base.Joiner com.google.common.base.Joiner skipNulls()'>
-        <annotation name="org.jetbrains.annotations.NotNull" />
-    </item>
-</root>
\ No newline at end of file
diff --git a/libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/src/main/kotlin/helloWorld.kt b/libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/src/main/kotlin/helloWorld.kt
deleted file mode 100644
index a844b9fe15d..00000000000
--- a/libraries/tools/kotlin-gradle-plugin/src/test/resources/testProject/kdocProject/src/main/kotlin/helloWorld.kt
+++ /dev/null
@@ -1,15 +0,0 @@
-package demo
-
-/**
- * This is **bold**
- *
- * New paragraph using newlines.
- *
- * We can also support wiki links notation
- * for example [[List]] and [[Collection]] and [[java.util.Collection.size()]] will
- * generate hypertext links to classes
- */
-public class MyClass() {
-    val i: Int = 0
-}
-