Skip to content

Commit bb18761

Browse files
authored
Merge pull request #736 from AlexanderBartash/main
Attempt to fix Gradle 9 compatibility #735
2 parents df624c0 + bb02676 commit bb18761

File tree

3 files changed

+16
-6
lines changed

3 files changed

+16
-6
lines changed

plugins/convention-plugins/src/main/kotlin/PropertyOrEnv.kt

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,10 @@ internal fun Project.propertyOrEnv(key: String): String {
66
?: error("Didn't find any value for the key \"$key\" in Project properties or environment variables.")
77
}
88

9-
internal fun Project.propertyOrEnvOrNull(key: String): String? {
10-
return findProperty(key) as String? ?: System.getenv(key)
9+
internal fun Project.propertyOrEnvOrNull(
10+
key: String,
11+
nullIfEmpty: Boolean = true
12+
): String? {
13+
return (findProperty(key) as String?)?.takeUnless { nullIfEmpty && it.isEmpty() }
14+
?: System.getenv(key)?.takeUnless { nullIfEmpty && it.isEmpty() }
1115
}

plugins/convention-plugins/src/main/kotlin/gradle-plugin.gradle.kts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ plugins {
44
}
55

66
signing {
7+
isRequired = false
78
useInMemoryPgpKeys(
89
propertyOrEnvOrNull("GPG_key_id"),
910
propertyOrEnvOrNull("GPG_private_key") ?: return@signing,

plugins/core/src/main/kotlin/de/fayard/refreshVersions/core/RefreshVersionsCoreSetup.kt

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import de.fayard.refreshVersions.core.internal.setupVersionPlaceholdersResolving
1515
import org.gradle.api.artifacts.ExternalDependency
1616
import org.gradle.api.file.RegularFile
1717
import org.gradle.api.initialization.Settings
18-
import org.gradle.api.internal.artifacts.dependencies.DefaultClientModule
1918
import org.gradle.kotlin.dsl.apply
2019
import org.gradle.tooling.UnsupportedVersionException
2120
import org.gradle.util.GradleVersion
@@ -212,7 +211,7 @@ private fun setupPluginsVersionsResolution(
212211
val pluginVersion = requested.version ?: return@eachPlugin
213212
UsedPluginsTracker.pluginHasNoEntryInVersionsFile(
214213
settings = settings,
215-
dependency = pluginIdToDependency(pluginId, pluginVersion)
214+
dependency = pluginIdToDependency(settings, pluginId, pluginVersion)
216215
)
217216
return@eachPlugin
218217
}
@@ -248,5 +247,11 @@ internal fun pluginDependencyNotationToVersionKey(dependencyNotation: String): S
248247
else -> null
249248
}
250249

251-
internal fun pluginIdToDependency(pluginId: String, version: String): ExternalDependency =
252-
DefaultClientModule(pluginId, "$pluginId.gradle.plugin", version)
250+
internal fun pluginIdToDependency(
251+
settings: Settings,
252+
pluginId: String,
253+
version: String
254+
): ExternalDependency {
255+
val dependencyNotation = "$pluginId:$pluginId.gradle.plugin:$version"
256+
return settings.gradle.rootProject.buildscript.dependencies.create(dependencyNotation) as ExternalDependency
257+
}

0 commit comments

Comments
 (0)