You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by pa...@apache.org on 2022/09/21 03:33:14 UTC
[groovy] 02/05: Cleanup build-logic
This is an automated email from the ASF dual-hosted git repository.
paulk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/groovy.git
commit db4cb65491b5b2f3a7fc5f1fd96b6176ecd46844
Author: Goooler <wa...@gmail.com>
AuthorDate: Sat Sep 3 11:48:27 2022 +0800
Cleanup build-logic
---
.../org.apache.groovy-bad-practices-detection.gradle | 4 +++-
.../src/main/groovy/org.apache.groovy-base.gradle | 18 +++++++++---------
.../groovy/org.apache.groovy-published-library.gradle | 6 ++++--
.../apache/groovy/gradle/GroovyLibraryExtension.groovy | 5 ++---
.../groovy/org/apache/groovy/gradle/JarJarTask.groovy | 3 +--
.../apache/groovy/gradle/SharedConfiguration.groovy | 4 ++--
6 files changed, 21 insertions(+), 19 deletions(-)
diff --git a/build-logic/src/main/groovy/org.apache.groovy-bad-practices-detection.gradle b/build-logic/src/main/groovy/org.apache.groovy-bad-practices-detection.gradle
index d44211ec26..6cdb2ef56f 100644
--- a/build-logic/src/main/groovy/org.apache.groovy-bad-practices-detection.gradle
+++ b/build-logic/src/main/groovy/org.apache.groovy-bad-practices-detection.gradle
@@ -1,3 +1,5 @@
+import groovy.transform.CompileStatic
+
/*
* Licensed to the Apache Software Foundation (ASF) under one
* or more contributor license agreements. See the NOTICE file
@@ -42,7 +44,7 @@ allprojects { p ->
p.configurations.metaClass = new ContainerMetaclassClass(p.configurations.metaClass, checkResolution)
}
-@groovy.transform.CompileStatic
+@CompileStatic
class ContainerMetaclassClass extends DelegatingMetaClass {
Closure configuration
diff --git a/build-logic/src/main/groovy/org.apache.groovy-base.gradle b/build-logic/src/main/groovy/org.apache.groovy-base.gradle
index 06b3719779..8e14e7e042 100644
--- a/build-logic/src/main/groovy/org.apache.groovy-base.gradle
+++ b/build-logic/src/main/groovy/org.apache.groovy-base.gradle
@@ -322,14 +322,14 @@ afterEvaluate {
}
@CompileStatic
-Configuration createConsumableConfiguration(ObjectFactory objects,
- TaskContainer tasks,
- ConfigurationContainer container,
- String name,
- String usage,
- String artifactTask,
- List<String> capabilities,
- int targetJvmVersion) {
+static Configuration createConsumableConfiguration(ObjectFactory objects,
+ TaskContainer tasks,
+ ConfigurationContainer container,
+ String name,
+ String usage,
+ String artifactTask,
+ List<String> capabilities,
+ int targetJvmVersion) {
container.create(name) { Configuration cnf ->
cnf.canBeConsumed = true
cnf.canBeResolved = false
@@ -348,4 +348,4 @@ Configuration createConsumableConfiguration(ObjectFactory objects,
}
}
}
-}
\ No newline at end of file
+}
diff --git a/build-logic/src/main/groovy/org.apache.groovy-published-library.gradle b/build-logic/src/main/groovy/org.apache.groovy-published-library.gradle
index 3acc6946fb..d993eb950f 100644
--- a/build-logic/src/main/groovy/org.apache.groovy-published-library.gradle
+++ b/build-logic/src/main/groovy/org.apache.groovy-published-library.gradle
@@ -1,3 +1,5 @@
+import groovy.swing.SwingBuilder
+
plugins {
id 'maven-publish'
id 'signing'
@@ -826,7 +828,7 @@ gradle.taskGraph.whenReady { taskGraph ->
def promptUser(String prompt) {
def response = ''
if (System.console() == null) {
- new groovy.swing.SwingBuilder().edt {
+ new SwingBuilder().edt {
dialog(modal: true, // pause build
title: 'Response required', // dialog title
alwaysOnTop: true,
@@ -853,4 +855,4 @@ def promptUser(String prompt) {
throw new InvalidUserDataException("Null response detected!")
}
response
-}
\ No newline at end of file
+}
diff --git a/build-logic/src/main/groovy/org/apache/groovy/gradle/GroovyLibraryExtension.groovy b/build-logic/src/main/groovy/org/apache/groovy/gradle/GroovyLibraryExtension.groovy
index 74468cf287..cb87c31fe4 100644
--- a/build-logic/src/main/groovy/org/apache/groovy/gradle/GroovyLibraryExtension.groovy
+++ b/build-logic/src/main/groovy/org/apache/groovy/gradle/GroovyLibraryExtension.groovy
@@ -42,7 +42,6 @@ import org.gradle.api.plugins.JavaPluginExtension
import org.gradle.api.provider.ListProperty
import org.gradle.api.provider.Property
import org.gradle.api.tasks.TaskContainer
-import org.gradle.api.tasks.TaskProvider
import org.gradle.api.tasks.bundling.Jar
import javax.inject.Inject
@@ -111,7 +110,7 @@ class GroovyLibraryExtension {
) {
grooid.set(true)
def grooidJar = tasks.register("grooidJar", JarJarTask) {
- TaskProvider<JarJarTask> jarjar = (TaskProvider<JarJarTask>) tasks.named("jarjar")
+ def jarjar = tasks.named("jarjar", JarJarTask)
it.dependsOn(jarjar)
it.from.set(jarjar.flatMap { it.outputFile })
if (librariesToRepackage) {
@@ -130,7 +129,7 @@ class GroovyLibraryExtension {
it.createManifest = false
it.includedResources = resources
it.outputFile.set(layout.buildDirectory.file(
- tasks.named('jar').map { Jar jar ->
+ tasks.named('jar', Jar).map { jar ->
"libs/${jar.archiveBaseName.get()}-${jar.archiveVersion.get()}-grooid.jar"
}
))
diff --git a/build-logic/src/main/groovy/org/apache/groovy/gradle/JarJarTask.groovy b/build-logic/src/main/groovy/org/apache/groovy/gradle/JarJarTask.groovy
index 369fdf21a4..d80107cd63 100644
--- a/build-logic/src/main/groovy/org/apache/groovy/gradle/JarJarTask.groovy
+++ b/build-logic/src/main/groovy/org/apache/groovy/gradle/JarJarTask.groovy
@@ -191,5 +191,4 @@ class JarJarTask extends DefaultTask {
private static String baseName(File file) {
file.name.substring(0, file.name.lastIndexOf('-'))
}
-
-}
\ No newline at end of file
+}
diff --git a/build-logic/src/main/groovy/org/apache/groovy/gradle/SharedConfiguration.groovy b/build-logic/src/main/groovy/org/apache/groovy/gradle/SharedConfiguration.groovy
index b03304642e..f7acf93ac8 100644
--- a/build-logic/src/main/groovy/org/apache/groovy/gradle/SharedConfiguration.groovy
+++ b/build-logic/src/main/groovy/org/apache/groovy/gradle/SharedConfiguration.groovy
@@ -114,7 +114,7 @@ class SharedConfiguration {
logger.lifecycle "ArtifactoryUser user: ${username.getOrElse("not defined")}"
}
- private Provider<RegularFile> artifactoryFile(ProviderFactory providers, ProjectLayout layout) {
+ private static Provider<RegularFile> artifactoryFile(ProviderFactory providers, ProjectLayout layout) {
providers.provider {
// try to read artifactory.properties
Directory base = layout.projectDirectory
@@ -127,7 +127,7 @@ class SharedConfiguration {
}
}
- private Provider<String> provider(ProviderFactory providers, Provider<Properties> properties, String propertyName, String gradlePropertyName, String envVarName) {
+ private static Provider<String> provider(ProviderFactory providers, Provider<Properties> properties, String propertyName, String gradlePropertyName, String envVarName) {
return providers.gradleProperty(gradlePropertyName).forUseAtConfigurationTime()
.orElse(providers.environmentVariable(envVarName).forUseAtConfigurationTime())
.orElse(properties.map { it.getProperty(propertyName) })