Update to stable kotlin dsl

README.md

@@ -53,8 +53,8 @@ buildscript {
 Stand-alone examples are available for each of gradle's supported languages.
  * [Groovy](https://github.com/google/protobuf-gradle-plugin/tree/master/examples/exampleProject) ___(Default)___
    * Run `../../gradlew build` under the example directory to test it out.
- * [Kotlin (experimental)](https://github.com/google/protobuf-gradle-plugin/tree/master/examples/exampleKotlinDslProject)
-   * Run `./gradlew build` under the Kotlin example directory to test it out. This example is set up with Gradle 4.10, the minimum required version.
+ * [Kotlin](https://github.com/google/protobuf-gradle-plugin/tree/master/examples/exampleKotlinDslProject) ___(Experimental)___
+   * Run `./gradlew build` under the Kotlin example directory to test it out. This example is set up with Gradle 5.0, the minimum required version.
 
 
 Directories that start with `testProject` can also serve as usage

build.gradle

@@ -32,14 +32,17 @@ buildscript {
         classpath 'com.jfrog.bintray.gradle:gradle-bintray-plugin:1.6'
         classpath "com.gradle.publish:plugin-publish-plugin:0.9.7"
         classpath "com.github.ben-manes:gradle-versions-plugin:0.12.0"
-        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:1.2.0"
+        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.10"
     }
 }
 
 repositories {
     maven { url "https://plugins.gradle.org/m2/" }
     maven { url "https://repo.gradle.org/gradle/libs-releases-local/" }
     google()
+    // Needed for resolving 'kotlinx-metadata-jvm'
+    // A transitive dependency of gradle Kotlin DSL
+    maven { url "https://kotlin.bintray.com/kotlinx/" }
 }
 
 configurations {
@@ -68,7 +71,7 @@ task createClasspathManifest {
 dependencies {
     compileOnly gradleApi()
     compileOnly localGroovy()
-    compileOnly "org.gradle:gradle-kotlin-dsl:1.0-rc-6"
+    compileOnly "org.gradle:gradle-kotlin-dsl:1.0.4"
 
     compile 'com.google.guava:guava:18.0'
     compile 'com.google.gradle:osdetector-gradle-plugin:1.4.0'
@@ -83,6 +86,8 @@ dependencies {
     }
     testCompile 'commons-io:commons-io:2.5'
 
+    // This kotlin version needs to be compatible with the 
+    // android plugin being used in the test runtime.
     testProjectRuntime "org.jetbrains.kotlin:kotlin-gradle-plugin:1.2.0"
 
     // Add android plugin to the test classpath, so that we can jump into class definitions,

examples/exampleKotlinDslProject/build.gradle.kts

@@ -7,7 +7,7 @@ import org.gradle.kotlin.dsl.provider.gradleKotlinDslOf
 plugins {
     java
     idea
-    id("com.google.protobuf") version "0.8.7-SNAPSHOT"
+    id("com.google.protobuf") version "0.8.8-SNAPSHOT"
 }
 
 repositories {

examples/exampleKotlinDslProject/gradle/wrapper/gradle-wrapper.properties

@@ -1,5 +1,5 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-4.10.2-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-5.0-bin.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

src/main/kotlin/com/google/protobuf/gradle/ProtobufConfiguratorExts.kt

@@ -4,61 +4,170 @@ import org.gradle.api.NamedDomainObjectContainer
 import org.gradle.api.Project
 import org.gradle.api.plugins.ExtensionAware
 import org.gradle.api.tasks.SourceSet
-import org.gradle.kotlin.dsl.*
-
+import org.gradle.kotlin.dsl.NamedDomainObjectContainerScope
+import org.gradle.kotlin.dsl.closureOf
+import org.gradle.kotlin.dsl.get
 
+/**
+ * Applies the supplied action to the project's instance of [ProtobufConfigurator].
+ *
+ * @since 0.8.7
+ * @usage
+ * ```
+ * protobuf {
+ *     ...
+ *     generatedFilesBaseDir = files(...)
+ * }
+ * ```
+ *
+ * @receiver [Project] The project for which the plugin configuration will be applied
+ * @param action A configuration lambda to apply on a receiver of type [ProtobufConfigurator]
+ *
+ * @return [Unit]
+ */
 fun Project.protobuf(action: ProtobufConfigurator.()->Unit) {
     project.convention.getPlugin(ProtobufConvention::class.java).protobuf.apply(action)
 }
 
+/**
+ * Applies the supplied action to the [ProtobufSourceDirectorySet] extension on
+ * a receiver of type [SourceSet]
+ *
+ * @since 0.8.7
+ * @usage
+ * ```
+ * sourceSets {
+ *     create("sample") {
+ *         proto {
+ *             srcDir("src/sample/protobuf")
+ *         }
+ *     }
+ * }
+ * ```
+ *
+ * @receiver [SourceSet] The source set for which the [ProtobufSourceDirectorySet] extension
+ * will be configured
+ *
+ * @param action A configuration lambda to apply on a receiver of type [ProtobufSourceDirectorySet]
+ * @return [Unit]
+ */
 fun SourceSet.proto(action: ProtobufSourceDirectorySet.() -> Unit) {
     (this as? ExtensionAware)
         ?.extensions
         ?.getByType(ProtobufSourceDirectorySet::class.java)
         ?.apply(action)
 }
 
-fun ProtobufConfigurator.protoc(closure: ExecutableLocator.() -> Unit) {
-    protoc(closureOf(closure))
+/**
+ * Uses the supplied action to configure the [ExecutableLocator] for protoc.
+ *
+ * @since 0.8.7
+ * @usage
+ * ```
+ * protobuf {
+ *     protoc {
+ *         artifact = "com.google.protobuf:protoc:3.6.1"
+ *     }
+ * }
+ * ```
+ *
+ * @receiver [ProtobufConfigurator] The protobuf plugin configuration instance
+ * for the project.
+ *
+ * @param action A lambda with receiver of type [ExecutableLocator] used
+ * for configuring the locator for protoc
+ *
+ * @return [Unit]
+ */
+fun ProtobufConfigurator.protoc(action: ExecutableLocator.() -> Unit) {
+    protoc(closureOf(action))
 }
 
-fun ProtobufConfigurator.plugins(closure: NamedDomainObjectContainerScope<ExecutableLocator>.() -> Unit) {
+fun ProtobufConfigurator.plugins(action: NamedDomainObjectContainerScope<ExecutableLocator>.() -> Unit) {
     plugins(closureOf<NamedDomainObjectContainer<ExecutableLocator>> {
-        this(closure)
+        NamedDomainObjectContainerScope.of(this).action()
     })
 }
 
-fun <T : Any> NamedDomainObjectContainerScope<T>.id(id: String, closure: (T.() -> Unit)? = null) {
-    closure?.let { create(id, closureOf(it)) }
-        ?: create(id)
-}
-
-fun <T : Any> NamedDomainObjectContainerScope<T>.remove(id: String) {
-    remove(this[id])
-}
-
-fun ProtobufConfigurator.generateProtoTasks(closure: ProtobufConfigurator.GenerateProtoTaskCollection.()->Unit) {
-    generateProtoTasks(closureOf(closure))
+fun ProtobufConfigurator.generateProtoTasks(action: ProtobufConfigurator.GenerateProtoTaskCollection.()->Unit) {
+    generateProtoTasks(closureOf(action))
 }
 
-fun GenerateProtoTask.builtins(closure: NamedDomainObjectContainerScope<GenerateProtoTask.PluginOptions>.()->Unit) {
+fun GenerateProtoTask.builtins(action: NamedDomainObjectContainerScope<GenerateProtoTask.PluginOptions>.()->Unit) {
     builtins(closureOf<NamedDomainObjectContainer<GenerateProtoTask.PluginOptions>> {
-        this(closure)
+        NamedDomainObjectContainerScope.of(this).action()
     })
 }
 
-fun GenerateProtoTask.plugins(closure: NamedDomainObjectContainerScope<GenerateProtoTask.PluginOptions>.()-> Unit) {
+fun GenerateProtoTask.plugins(action: NamedDomainObjectContainerScope<GenerateProtoTask.PluginOptions>.()-> Unit) {
     plugins(closureOf<NamedDomainObjectContainer<GenerateProtoTask.PluginOptions>> {
-        this(closure)
+        NamedDomainObjectContainerScope.of(this).action()
     })
 }
 
+/**
+ * An extension for creating and configuring the elements of an instance of [NamedDomainObjectContainer].
+ *
+ * @since 0.8.7
+ * @usage
+ * ```
+ * protobuf {
+ *     plugins {
+ *         id("grpc") {
+ *             artifact = "io.grpc:protoc-gen-grpc-java:1.15.1"
+ *         }
+ *     }
+ * }
+ * ```
+ *
+ * @receiver [NamedDomainObjectContainerScope] The scope of the [NamedDomainObjectContainer]
+ * on which to create or configure an element.
+ *
+ * @param id The string id of the element to create or configure.
+ * @param action An optional action that will be applied to the element instance.
+ *
+ * @return [Unit]
+ */
+fun <T : Any> NamedDomainObjectContainerScope<T>.id(id: String, action: (T.() -> Unit)? = null) {
+    action?.let { create(id, closureOf(it)) }
+            ?: create(id)
+}
+
+
+/**
+ * An extension for removing an element by id on an instance of [NamedDomainObjectContainer].
+ *
+ * @since 0.8.7
+ * @usage
+ * ```
+ * protobuf {
+ *     generateProtoTasks {
+ *         ofSourceSet("main").forEach {
+ *             it.builtins {
+ *                 remove("java")
+ *             }
+ *         }
+ *     }
+ * }
+ * ```
+ *
+ * @receiver [NamedDomainObjectContainerScope] The scope of the [NamedDomainObjectContainer]
+ * on which to remove an element.
+ *
+ * @param id The string id of the element to remove.
+ *
+ * @return [Unit]
+ */
+fun <T : Any> NamedDomainObjectContainerScope<T>.remove(id: String) {
+    remove(this[id])
+}
+
 /**
  * The method generatorProtoTasks applies the supplied closure to the
  * instance of [ProtobufConfigurator.GenerateProtoTaskCollection].
  *
  * Since [ProtobufConfigurator.JavaGenerateProtoTaskCollection] and [ProtobufConfigurator.AndroidGenerateProtoTaskCollection]
- * each have unique methods, and only one instance in allocated per project, we need a way to statically resolve the
+ * each have unique methods, and only one instance is allocated per project, we need a way to statically resolve the
  * available methods. This is a necessity since Kotlin does not have any dynamic method resolution capabilities.
  */
 

src/test/groovy/com/google/protobuf/gradle/ProtobufKotlinDslPluginTest.groovy

@@ -9,7 +9,7 @@ import spock.lang.Specification
  * Unit tests for kotlin dsl extensions.
  */
 class ProtobufKotlinDslPluginTest extends Specification {
-  private static final List<String> GRADLE_VERSIONS = ["4.10"]
+  private static final List<String> GRADLE_VERSIONS = ["5.0"]
 
   void "testProjectKotlinDsl should be successfully executed (java-only project)"() {
     given: "project from testProjectKotlinDslBase"

testProjectKotlinDslBase/build.gradle.kts

@@ -100,7 +100,7 @@ tasks {
             from(sourceSet.output)
 
             val compileTaskName = sourceSet.getCompileTaskName("java")
-            dependsOn(tasks.getByName(compileTaskName))
+            dependsOn(project.tasks.getByName(compileTaskName))
         }
     }