Revert "Gradle 6.2 as minimum (#251)"

This reverts commit 535705a. Workaround for "null" values in credentails. See: #267
gradle-nexus · Sep 1, 2023 · 802a328 · 802a328
1 parent 2a18e12
commit 802a328
Show file tree

Hide file tree

Showing 7 changed files with 41 additions and 62 deletions.
diff --git a/.github/workflows/java-versions.yml b/.github/workflows/java-versions.yml
@@ -29,7 +29,7 @@ jobs:
       - name: Reduce number of Gradle regressions builds
         run: |
           # Keep only the last Gradle version per Stutter matrix; for each line this transformation is done:
-          # java11=6.2.2,6.9.4,7.0.2,7.6.1,8.0.2,8.1.1,8.2-rc-1
+          # java11=6.0.1,6.9.4,7.0.2,7.6.1,8.0.2,8.1.1,8.2-rc-1
           # ->
           # java11=8.2-rc-1
           # The trick is that \2 will greedily eat everything before the last comma.

diff --git a/README.md b/README.md
@@ -12,7 +12,7 @@ This plugin is intended as a replacement of the [Gradle Nexus Staging Plugin](ht
 
 ### Applying the plugin
 
-The plugin must be applied to the root project and requires Gradle 6.2 or later. It is important to
+The plugin must be applied to the root project and requires Gradle 6.0 or later. It is important to
 set the group and the version to the root project, so the plugin can detect if it is a snapshot
 version or not in order to select the correct repository where artifacts will be published.
 

diff --git a/build.gradle.kts b/build.gradle.kts
@@ -110,15 +110,15 @@ stutter {
                 languageVersion = JavaLanguageVersion.of(8)
             }
             gradleVersions {
-                compatibleRange("6.2")
+                compatibleRange("6.0")
             }
         }
         register("java11") {
             javaToolchain {
                 languageVersion = JavaLanguageVersion.of(11)
             }
             gradleVersions {
-                compatibleRange("6.2")
+                compatibleRange("6.0")
             }
         }
         register("java17") {
@@ -165,7 +165,7 @@ sourceSets {
 }
 
 kotlin.target.compilations.configureEach {
-    // Supporting Gradle 6.2+ needs to use Kotlin 1.3.
+    // Supporting Gradle 6.0+ needs to use Kotlin 1.3.
     // See https://docs.gradle.org/current/userguide/compatibility.html
     // For future maintainer: Kotlin 1.9.0 dropped support for Kotlin 1.3, it'll only support 1.4+.
     // This means Gradle 7.0 will be the lowest supportable version for plugins.

diff --git a/src/main/kotlin/io/github/gradlenexus/publishplugin/FindStagingRepository.kt b/src/main/kotlin/io/github/gradlenexus/publishplugin/FindStagingRepository.kt
@@ -16,7 +16,7 @@
 
 package io.github.gradlenexus.publishplugin
 
-import io.github.gradlenexus.publishplugin.internal.StagingRepositoryDescriptorRegistryBuildService
+import io.github.gradlenexus.publishplugin.internal.InvalidatingStagingRepositoryDescriptorRegistry
 import io.github.gradlenexus.publishplugin.internal.determineStagingProfileId
 import org.gradle.api.Incubating
 import org.gradle.api.provider.Property
@@ -29,8 +29,7 @@ import org.gradle.api.tasks.TaskAction
 abstract class FindStagingRepository : AbstractNexusStagingRepositoryTask() {
 
     @get:Internal
-    // TODO use @ServiceReference instead of @Internal when minimum is Gradle 8.0.
-    abstract val registry: Property<StagingRepositoryDescriptorRegistryBuildService>
+    abstract val registry: Property<InvalidatingStagingRepositoryDescriptorRegistry>
 
     @get:Optional
     @get:Input
@@ -59,6 +58,6 @@ abstract class FindStagingRepository : AbstractNexusStagingRepositoryTask() {
         val descriptor = client.findStagingRepository(stagingProfileId, Regex(descriptionRegex))
         logger.lifecycle("Staging repository for {} at {}, stagingProfileId '{}', descriptionRegex '{}' is '{}'", repository.name, serverUrl, stagingProfileId, descriptionRegex, descriptor.stagingRepositoryId)
         stagingRepositoryId.set(descriptor.stagingRepositoryId)
-        registry.get().registry[repository.name] = descriptor
+        registry.get()[repository.name] = descriptor
     }
 }
diff --git a/src/main/kotlin/io/github/gradlenexus/publishplugin/InitializeNexusStagingRepository.kt b/src/main/kotlin/io/github/gradlenexus/publishplugin/InitializeNexusStagingRepository.kt
@@ -16,7 +16,7 @@
 
 package io.github.gradlenexus.publishplugin
 
-import io.github.gradlenexus.publishplugin.internal.StagingRepositoryDescriptorRegistryBuildService
+import io.github.gradlenexus.publishplugin.internal.InvalidatingStagingRepositoryDescriptorRegistry
 import io.github.gradlenexus.publishplugin.internal.determineStagingProfileId
 import okhttp3.HttpUrl
 import org.gradle.api.provider.Property
@@ -28,8 +28,7 @@ import org.gradle.api.tasks.TaskAction
 abstract class InitializeNexusStagingRepository : AbstractNexusStagingRepositoryTask() {
 
     @get:Internal
-    // TODO use @ServiceReference instead of @Internal when minimum is Gradle 8.0.
-    abstract val registry: Property<StagingRepositoryDescriptorRegistryBuildService>
+    abstract val registry: Property<InvalidatingStagingRepositoryDescriptorRegistry>
 
     @get:Optional
     @get:Input
@@ -45,6 +44,6 @@ abstract class InitializeNexusStagingRepository : AbstractNexusStagingRepository
         val descriptor = client.createStagingRepository(stagingProfileId, repositoryDescription.get())
         val consumerUrl = HttpUrl.get(serverUrl)!!.newBuilder().addEncodedPathSegments("repositories/${descriptor.stagingRepositoryId}/content/").build()
         logger.lifecycle("Created staging repository '{}' at {}", descriptor.stagingRepositoryId, consumerUrl)
-        registry.get().registry[repository.name] = descriptor
+        registry.get()[repository.name] = descriptor
     }
 }
diff --git a/src/main/kotlin/io/github/gradlenexus/publishplugin/NexusPublishPlugin.kt b/src/main/kotlin/io/github/gradlenexus/publishplugin/NexusPublishPlugin.kt
@@ -17,6 +17,7 @@
 package io.github.gradlenexus.publishplugin
 
 import io.github.gradlenexus.publishplugin.NexusRepository.PublicationType
+import io.github.gradlenexus.publishplugin.internal.InvalidatingStagingRepositoryDescriptorRegistry
 import io.github.gradlenexus.publishplugin.internal.StagingRepositoryDescriptorRegistryBuildService
 import org.gradle.api.Action
 import org.gradle.api.Plugin
@@ -53,8 +54,8 @@ class NexusPublishPlugin : Plugin<Project> {
             "Plugin must be applied to the root project but was applied to ${project.path}"
         }
 
-        require(GradleVersion.current() >= GradleVersion.version("6.2")) {
-            "io.github.gradle-nexus.publish-plugin requires Gradle version 6.2+"
+        require(GradleVersion.current() >= GradleVersion.version("6.0")) {
+            "The plugin requires Gradle version 6.0+"
         }
 
         val registry = createRegistry(project)
@@ -77,18 +78,19 @@ class NexusPublishPlugin : Plugin<Project> {
         }
     }
 
-    private fun createRegistry(rootProject: Project): Provider<StagingRepositoryDescriptorRegistryBuildService> =
-        rootProject.gradle.sharedServices.registerIfAbsent(
-            "stagingRepositoryUrlRegistry",
-            StagingRepositoryDescriptorRegistryBuildService::class,
-            Action { }
-        )
+    private fun createRegistry(rootProject: Project): Provider<InvalidatingStagingRepositoryDescriptorRegistry> {
+        if (GradleVersion.current() >= GradleVersion.version("6.1")) {
+            return rootProject.gradle.sharedServices.registerIfAbsent(
+                "stagingRepositoryUrlRegistry",
+                StagingRepositoryDescriptorRegistryBuildService::class,
+                Action { }
+            ).map { it.registry }
+        }
+        val registry = InvalidatingStagingRepositoryDescriptorRegistry()
+        return rootProject.provider { registry }
+    }
 
-    private fun configureNexusTasks(
-        rootProject: Project,
-        extension: NexusPublishExtension,
-        registryService: Provider<StagingRepositoryDescriptorRegistryBuildService>
-    ) {
+    private fun configureNexusTasks(rootProject: Project, extension: NexusPublishExtension, registry: Provider<InvalidatingStagingRepositoryDescriptorRegistry>) {
         rootProject.tasks.withType(AbstractNexusStagingRepositoryTask::class.java).configureEach {
             clientTimeout.convention(extension.clientTimeout)
             connectTimeout.convention(extension.connectTimeout)
@@ -98,14 +100,13 @@ class NexusPublishPlugin : Plugin<Project> {
         }
         rootProject.tasks.withType(AbstractTransitionNexusStagingRepositoryTask::class.java).configureEach {
             transitionCheckOptions.convention(extension.transitionCheckOptions)
-            usesService(registryService)
-            stagingRepositoryId.convention(registryService.map { it.registry[repository.get().name].stagingRepositoryId })
+            stagingRepositoryId.convention(registry.map { it[repository.get().name].stagingRepositoryId })
         }
         extension.repositories.all {
-            username.convention(rootProject.providers.gradleProperty("${name}Username").forUseAtConfigurationTimeCompat())
-            password.convention(rootProject.providers.gradleProperty("${name}Password").forUseAtConfigurationTimeCompat())
+            username.convention(rootProject.provider { rootProject.findProperty("${name}Username") as? String })
+            password.convention(rootProject.provider { rootProject.findProperty("${name}Password") as? String })
             publicationType.convention(PublicationType.MAVEN)
-            configureRepositoryTasks(rootProject.tasks, extension, this, registryService)
+            configureRepositoryTasks(rootProject.tasks, extension, this, registry)
         }
         extension.repositories.whenObjectRemoved {
             rootProject.tasks.named("initialize${capitalizedName}StagingRepository").configure {
@@ -135,7 +136,7 @@ class NexusPublishPlugin : Plugin<Project> {
         tasks: TaskContainer,
         extension: NexusPublishExtension,
         repo: NexusRepository,
-        registryService: Provider<StagingRepositoryDescriptorRegistryBuildService>
+        registryProvider: Provider<InvalidatingStagingRepositoryDescriptorRegistry>
     ) {
         @Suppress("UNUSED_VARIABLE") // Keep it consistent.
         val retrieveStagingProfileTask = tasks.register<RetrieveStagingProfile>(
@@ -153,8 +154,7 @@ class NexusPublishPlugin : Plugin<Project> {
         ) {
             group = PublishingPlugin.PUBLISH_TASK_GROUP
             description = "Initializes the staging repository in '${repo.name}' Nexus instance."
-            registry.set(registryService)
-            usesService(registryService)
+            registry.set(registryProvider)
             repository.convention(repo)
             packageGroup.convention(extension.packageGroup)
         }
@@ -163,8 +163,7 @@ class NexusPublishPlugin : Plugin<Project> {
         ) {
             group = PublishingPlugin.PUBLISH_TASK_GROUP
             description = "Finds the staging repository for ${repo.name}"
-            registry.set(registryService)
-            usesService(registryService)
+            registry.set(registryProvider)
             repository.convention(repo)
             packageGroup.convention(extension.packageGroup)
             descriptionRegex.convention(extension.repositoryDescription.map { "\\b" + Regex.escape(it) + "(\\s|$)" })
@@ -204,11 +203,7 @@ class NexusPublishPlugin : Plugin<Project> {
         }
     }
 
-    private fun configurePublishingForAllProjects(
-        rootProject: Project,
-        extension: NexusPublishExtension,
-        registry: Provider<StagingRepositoryDescriptorRegistryBuildService>
-    ) {
+    private fun configurePublishingForAllProjects(rootProject: Project, extension: NexusPublishExtension, registry: Provider<InvalidatingStagingRepositoryDescriptorRegistry>) {
         rootProject.afterEvaluate {
             allprojects {
                 val publishingProject = this
@@ -247,7 +242,7 @@ class NexusPublishPlugin : Plugin<Project> {
     private fun addPublicationRepositories(
         project: Project,
         extension: NexusPublishExtension,
-        registry: Provider<StagingRepositoryDescriptorRegistryBuildService>
+        registry: Provider<InvalidatingStagingRepositoryDescriptorRegistry>
     ): Map<NexusRepository, ArtifactRepository> = extension.repositories.associateWith { nexusRepo ->
         createArtifactRepository(nexusRepo.publicationType.get(), project, nexusRepo, extension, registry)
     }
@@ -257,7 +252,7 @@ class NexusPublishPlugin : Plugin<Project> {
         project: Project,
         nexusRepo: NexusRepository,
         extension: NexusPublishExtension,
-        registry: Provider<StagingRepositoryDescriptorRegistryBuildService>
+        registry: Provider<InvalidatingStagingRepositoryDescriptorRegistry>
     ): ArtifactRepository = when (publicationType) {
         PublicationType.MAVEN -> project.theExtension<PublishingExtension>().repositories.maven {
             configureArtifactRepo(nexusRepo, extension, registry, false)
@@ -276,7 +271,7 @@ class NexusPublishPlugin : Plugin<Project> {
     private fun <T> T.configureArtifactRepo(
         nexusRepo: NexusRepository,
         extension: NexusPublishExtension,
-        registry: Provider<StagingRepositoryDescriptorRegistryBuildService>,
+        registry: Provider<InvalidatingStagingRepositoryDescriptorRegistry>,
         provideFallback: Boolean
     ) where T : UrlArtifactRepository, T : ArtifactRepository, T : AuthenticationSupported {
         name = nexusRepo.name
@@ -332,13 +327,13 @@ class NexusPublishPlugin : Plugin<Project> {
     private fun getRepoUrl(
         nexusRepo: NexusRepository,
         extension: NexusPublishExtension,
-        registry: Provider<StagingRepositoryDescriptorRegistryBuildService>,
+        registry: Provider<InvalidatingStagingRepositoryDescriptorRegistry>,
         provideFallback: Boolean,
         artifactRepo: ArtifactRepository
     ): Provider<URI> =
         extension.useStaging.flatMap { useStaging ->
             if (useStaging) {
-                registry.map { it.registry }.map { descriptorRegistry ->
+                registry.map { descriptorRegistry ->
                     if (provideFallback) {
                         descriptorRegistry.invalidateLater(nexusRepo.name, artifactRepo)
                         descriptorRegistry.tryGet(nexusRepo.name)?.stagingRepositoryUrl ?: nexusRepo.nexusUrl.get()
@@ -380,17 +375,3 @@ private inline fun <reified T : Any> Project.theExtension(): T =
         this.extensions.findByType(it)
             ?: error("The plugin cannot be applied without the publishing plugin")
     }
-
-private fun <T> Provider<T>.forUseAtConfigurationTimeCompat(): Provider<T> =
-    if (GradleVersion.current() < GradleVersion.version("6.5")) {
-        // Gradle < 6.5 doesn't have this function.
-        this
-    } else if (GradleVersion.current() < GradleVersion.version("7.4")) {
-        // Gradle 6.5 - 7.3 requires this function to be called.
-        @Suppress("DEPRECATION")
-        this.forUseAtConfigurationTime()
-    } else {
-        // Gradle >= 7.4 deprecated this function in favor of not calling it (became no-op, and will eventually nag).
-        // https://docs.gradle.org/current/userguide/upgrading_version_7.html#for_use_at_configuration_time_deprecation
-        this
-    }
diff --git a/stutter.lockfile b/stutter.lockfile
@@ -1,4 +1,4 @@
 # DO NOT MODIFY: Generated by Stutter plugin.
-java11=6.2.2,6.9.4,7.0.2,7.6.2,8.0.2,8.2.1
+java11=6.0.1,6.9.4,7.0.2,7.6.2,8.0.2,8.2.1
 java17=7.3.3,7.6.2,8.0.2,8.2.1
-java8=6.2.2,6.9.4,7.0.2,7.6.2,8.0.2,8.2.1
+java8=6.0.1,6.9.4,7.0.2,7.6.2,8.0.2,8.2.1