This is an automated email from the ASF dual-hosted git repository.

vladimirsitnikov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/master by this push:
     new 3f049ee  Update Gradle: 6.3 -> 6.5
3f049ee is described below

commit 3f049eed72cebb916883912affed19154a2bbcfa
Author: Vladimir Sitnikov <sitnikov.vladi...@gmail.com>
AuthorDate: Fri Jun 12 21:41:48 2020 +0300

    Update Gradle: 6.3 -> 6.5
---
 babel/build.gradle.kts                                  |  2 +-
 .../org/apache/calcite/buildtools/javacc/JavaCCTask.kt  | 17 +++++++----------
 core/build.gradle.kts                                   |  4 ++--
 gradle/wrapper/gradle-wrapper.properties                |  4 ++--
 server/build.gradle.kts                                 |  2 +-
 5 files changed, 13 insertions(+), 16 deletions(-)

diff --git a/babel/build.gradle.kts b/babel/build.gradle.kts
index ee24594..091bcd8 100644
--- a/babel/build.gradle.kts
+++ b/babel/build.gradle.kts
@@ -46,7 +46,7 @@ val javaCCMain by 
tasks.registering(org.apache.calcite.buildtools.javacc.JavaCCT
     dependsOn(fmppMain)
     lookAhead.set(2)
     val parserFile = fmppMain.map {
-        it.output.asFileTree.matching { include("**/Parser.jj") }.singleFile
+        it.output.asFileTree.matching { include("**/Parser.jj") 
}.singleOrNull().toPlatformType()
     }
     inputFile.set(parserFile)
     packageName.set("org.apache.calcite.sql.parser.babel")
diff --git 
a/buildSrc/subprojects/javacc/src/main/kotlin/org/apache/calcite/buildtools/javacc/JavaCCTask.kt
 
b/buildSrc/subprojects/javacc/src/main/kotlin/org/apache/calcite/buildtools/javacc/JavaCCTask.kt
index 4de0ea8..ced2c71 100644
--- 
a/buildSrc/subprojects/javacc/src/main/kotlin/org/apache/calcite/buildtools/javacc/JavaCCTask.kt
+++ 
b/buildSrc/subprojects/javacc/src/main/kotlin/org/apache/calcite/buildtools/javacc/JavaCCTask.kt
@@ -18,6 +18,7 @@
 package org.apache.calcite.buildtools.javacc
 
 import java.io.File
+import java.util.function.Function
 import javax.inject.Inject
 import org.gradle.api.DefaultTask
 import org.gradle.api.artifacts.Configuration
@@ -25,7 +26,6 @@ import org.gradle.api.model.ObjectFactory
 import org.gradle.api.tasks.Classpath
 import org.gradle.api.tasks.Input
 import org.gradle.api.tasks.InputFile
-import org.gradle.api.tasks.Internal
 import org.gradle.api.tasks.OutputDirectory
 import org.gradle.api.tasks.TaskAction
 import org.gradle.kotlin.dsl.property
@@ -37,17 +37,14 @@ open class JavaCCTask @Inject constructor(
     val javaCCClasspath = objectFactory.property<Configuration>()
         
.convention(project.configurations.named(JavaCCPlugin.JAVACC_CLASSPATH_CONFIGURATION_NAME))
 
-    @Internal
+    @InputFile
     val inputFile = objectFactory.property<File>()
 
-    // See https://github.com/gradle/gradle/issues/12627
-    @get:InputFile
-    val actualInputFile: File? get() = try {
-        inputFile.get()
-    } catch (e: IllegalStateException) {
-        // This means Gradle queries property too early
-        null
-    }
+    /**
+     * This enables to cast a nullable value to a non-nullable one
+     * @see [https://github.com/gradle/gradle/issues/12627]
+     */
+    inline fun <reified T> T?.toPlatformType(): T = 
Function.identity<T>().apply(this)
 
     @Input
     val lookAhead = objectFactory.property<Int>().convention(1)
diff --git a/core/build.gradle.kts b/core/build.gradle.kts
index 6a5c103..c4faaac 100644
--- a/core/build.gradle.kts
+++ b/core/build.gradle.kts
@@ -159,7 +159,7 @@ val fmppMain by 
tasks.registering(org.apache.calcite.buildtools.fmpp.FmppTask::c
 val javaCCMain by 
tasks.registering(org.apache.calcite.buildtools.javacc.JavaCCTask::class) {
     dependsOn(fmppMain)
     val parserFile = fmppMain.map {
-        it.output.asFileTree.matching { include("**/Parser.jj") }.singleFile
+        it.output.asFileTree.matching { include("**/Parser.jj") 
}.singleOrNull().toPlatformType()
     }
     inputFile.set(parserFile)
     packageName.set("org.apache.calcite.sql.parser.impl")
@@ -173,7 +173,7 @@ val fmppTest by 
tasks.registering(org.apache.calcite.buildtools.fmpp.FmppTask::c
 val javaCCTest by 
tasks.registering(org.apache.calcite.buildtools.javacc.JavaCCTask::class) {
     dependsOn(fmppTest)
     val parserFile = fmppTest.map {
-        it.output.asFileTree.matching { include("**/Parser.jj") }.singleFile
+        it.output.asFileTree.matching { include("**/Parser.jj") 
}.singleOrNull().toPlatformType()
     }
     inputFile.set(parserFile)
     packageName.set("org.apache.calcite.sql.parser.parserextensiontesting")
diff --git a/gradle/wrapper/gradle-wrapper.properties 
b/gradle/wrapper/gradle-wrapper.properties
index 0db054b..2c65038 100644
--- a/gradle/wrapper/gradle-wrapper.properties
+++ b/gradle/wrapper/gradle-wrapper.properties
@@ -16,7 +16,7 @@
 #
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionSha256Sum=0f316a67b971b7b571dac7215dcf2591a30994b3450e0629925ffcfe2c68cc5c
-distributionUrl=https\://services.gradle.org/distributions/gradle-6.3-all.zip
+distributionSha256Sum=c9910513d0eed63cd8f5c7fec4cb4a05731144770104a0871234a4edc3ba3cef
+distributionUrl=https\://services.gradle.org/distributions/gradle-6.5-all.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists
diff --git a/server/build.gradle.kts b/server/build.gradle.kts
index 33a153d..58d09ff 100644
--- a/server/build.gradle.kts
+++ b/server/build.gradle.kts
@@ -45,7 +45,7 @@ val fmppMain by 
tasks.registering(org.apache.calcite.buildtools.fmpp.FmppTask::c
 val javaCCMain by 
tasks.registering(org.apache.calcite.buildtools.javacc.JavaCCTask::class) {
     dependsOn(fmppMain)
     val parserFile = fmppMain.map {
-        it.output.asFileTree.matching { include("**/Parser.jj") }.singleFile
+        it.output.asFileTree.matching { include("**/Parser.jj") 
}.singleOrNull().toPlatformType()
     }
     inputFile.set(parserFile)
     packageName.set("org.apache.calcite.sql.parser.ddl")

Reply via email to