You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by vl...@apache.org on 2020/08/30 13:40:22 UTC
[calcite] branch master updated (425f170 -> cf99329)
This is an automated email from the ASF dual-hosted git repository.
vladimirsitnikov pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git.
from 425f170 [CALCITE-4195] Cast between types with different collators must be evaluated as not monotonic
new 368e579 Update Checkstyle from 8.27 to 8.28 to support package-info files with imports
new cf99329 [CALCITE-4185] Remove dependency between :ubenchmark:checkstyleJmh and :ubenchmark:compileJava
The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
build.gradle.kts | 9 +++++++--
gradle.properties | 2 +-
2 files changed, 8 insertions(+), 3 deletions(-)
[calcite] 02/02: [CALCITE-4185] Remove dependency between
:ubenchmark:checkstyleJmh and :ubenchmark:compileJava
Posted by vl...@apache.org.
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
commit cf9932914bdeffbb1e96fa9464a476663eed6046
Author: Vladimir Sitnikov <si...@gmail.com>
AuthorDate: Sun Aug 30 16:38:45 2020 +0300
[CALCITE-4185] Remove dependency between :ubenchmark:checkstyleJmh and :ubenchmark:compileJava
---
build.gradle.kts | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/build.gradle.kts b/build.gradle.kts
index 748f5bc..59e883b 100644
--- a/build.gradle.kts
+++ b/build.gradle.kts
@@ -326,8 +326,6 @@ allprojects {
// On the other hand, supporessions.xml still analyzes the file, and
// then it recognizes it should suppress all the output.
excludeJavaCcGenerated()
- // Checkstyle 8.26 does not need classpath, see https://github.com/gradle/gradle/issues/14227
- classpath = files()
// Workaround for https://github.com/gradle/gradle/issues/13927
// Absolute paths must not be used as they defeat Gradle build cache
// Unfortunately, Gradle passes only config_loc variable by default, so we make
@@ -335,6 +333,13 @@ allprojects {
configProperties!!["cache_file"] =
buildDir.resolve("checkstyle/cacheFile").relativeTo(configLoc)
}
+ // afterEvaluate is to support late sourceSet addition (e.g. jmh sourceset)
+ afterEvaluate {
+ tasks.configureEach<Checkstyle> {
+ // Checkstyle 8.26 does not need classpath, see https://github.com/gradle/gradle/issues/14227
+ classpath = files()
+ }
+ }
}
if (!skipAutostyle || !skipCheckstyle) {
tasks.register("style") {
[calcite] 01/02: Update Checkstyle from 8.27 to 8.28 to support
package-info files with imports
Posted by vl...@apache.org.
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
commit 368e579fdc6bd83acf303055a59e2f6672fc48aa
Author: Vladimir Sitnikov <si...@gmail.com>
AuthorDate: Sun Aug 30 16:24:15 2020 +0300
Update Checkstyle from 8.27 to 8.28 to support package-info files with imports
see https://github.com/checkstyle/checkstyle/issues/7117
---
gradle.properties | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gradle.properties b/gradle.properties
index c3e25fb..867ef45 100644
--- a/gradle.properties
+++ b/gradle.properties
@@ -58,7 +58,7 @@ org.owasp.dependencycheck.version=5.2.2
# docker-maven-plugin.version=1.2.0
# Tools
-checkstyle.version=8.27
+checkstyle.version=8.28
spotbugs.version=3.1.11
# We support Guava versions as old as 14.0.1 (the version used by Hive)