Skip to content

Commit df2e5c6

Browse files
committed
Cleanup injected parameters
1 parent d2fd339 commit df2e5c6

3 files changed

Lines changed: 7 additions & 18 deletions

File tree

src/main/kotlin/io/github/sgtsilvio/gradle/oci/internal/dsl/OciExtensionImpl.kt

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,11 @@ import io.github.sgtsilvio.gradle.oci.mapping.OciImageMappingImpl
1212
import io.github.sgtsilvio.gradle.oci.platform.PlatformFilter
1313
import org.gradle.api.Action
1414
import org.gradle.api.Project
15-
import org.gradle.api.artifacts.ConfigurationContainer
1615
import org.gradle.api.file.ProjectLayout
1716
import org.gradle.api.model.ObjectFactory
1817
import org.gradle.api.plugins.jvm.JvmTestSuite
1918
import org.gradle.api.provider.Provider
2019
import org.gradle.api.provider.ProviderFactory
21-
import org.gradle.api.services.BuildServiceRegistry
2220
import org.gradle.api.tasks.TaskContainer
2321
import org.gradle.api.tasks.TaskProvider
2422
import org.gradle.api.tasks.testing.Test
@@ -35,9 +33,7 @@ internal abstract class OciExtensionImpl @Inject constructor(
3533
private val objectFactory: ObjectFactory,
3634
private val taskContainer: TaskContainer,
3735
private val projectLayout: ProjectLayout,
38-
buildServiceRegistry: BuildServiceRegistry,
3936
project: Project,
40-
configurationContainer: ConfigurationContainer,
4137
) : OciExtension {
4238

4339
final override val registries = objectFactory.newInstance<OciRegistriesImpl>()
@@ -60,7 +56,7 @@ internal abstract class OciExtensionImpl @Inject constructor(
6056
init {
6157
// eagerly realize imageDefinitions because they register configurations and tasks
6258
imageDefinitions.all {}
63-
setupProjectOciRegistries(buildServiceRegistry, project, configurationContainer, registries, imageMapping)
59+
setupProjectOciRegistries(project, registries, imageMapping)
6460
}
6561

6662
final override fun registries(configuration: Action<in OciRegistries>) = configuration.execute(registries)

src/main/kotlin/io/github/sgtsilvio/gradle/oci/internal/dsl/OciRegistriesImpl.kt

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import io.netty.channel.ChannelOption
1717
import io.netty.util.concurrent.FastThreadLocal
1818
import org.gradle.api.Action
1919
import org.gradle.api.Project
20-
import org.gradle.api.artifacts.ConfigurationContainer
2120
import org.gradle.api.artifacts.ResolvableDependencies
2221
import org.gradle.api.artifacts.dsl.RepositoryHandler
2322
import org.gradle.api.artifacts.repositories.InclusiveRepositoryContentDescriptor
@@ -52,8 +51,8 @@ internal val OCI_IMAGE_DISTRIBUTION_TYPES = arrayOf(OCI_IMAGE_DISTRIBUTION_TYPE,
5251
*/
5352
internal abstract class OciRegistriesImpl @Inject constructor(
5453
private val repositoryHandler: RepositoryHandler,
55-
private val objectFactory: ObjectFactory,
5654
private val providerFactory: ProviderFactory,
55+
private val objectFactory: ObjectFactory,
5756
) : OciRegistries {
5857
final override val list = objectFactory.namedDomainObjectList(OciRegistry::class)
5958
final override val repositoryPort: Property<Int> = objectFactory.property<Int>().convention(5123)
@@ -283,24 +282,18 @@ internal fun setupSettingsOciRegistries(
283282
)
284283
}
285284

286-
internal fun setupProjectOciRegistries(
287-
buildServiceRegistry: BuildServiceRegistry,
288-
project: Project,
289-
configurationContainer: ConfigurationContainer,
290-
registries: OciRegistries,
291-
imageMapping: OciImageMappingImpl,
292-
) {
285+
internal fun setupProjectOciRegistries(project: Project, registries: OciRegistries, imageMapping: OciImageMappingImpl) {
293286
var isOciRegistriesStarted = false
294-
configurationContainer.configureEach {
287+
project.configurations.configureEach {
295288
incoming.beforeResolve {
296289
if (!isOciRegistriesStarted && resolvesOciImages()) {
297290
isOciRegistriesStarted = true
298-
val settingsRegistration = buildServiceRegistry.registrations.findByName(SERVICE_BASE_NAME)
291+
val settingsRegistration = project.gradle.sharedServices.registrations.findByName(SERVICE_BASE_NAME)
299292
if (settingsRegistration != null) {
300293
(settingsRegistration.service.get() as OciRegistriesService).start()
301294
}
302295
OciRegistriesService(
303-
buildServiceRegistry,
296+
project.gradle.sharedServices,
304297
"$SERVICE_BASE_NAME-${project.path}",
305298
registries.list,
306299
registries.repositoryPort,

src/main/kotlin/io/github/sgtsilvio/gradle/oci/internal/dsl/OciSettingsExtensionImpl.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ import javax.inject.Inject
1717
*/
1818
internal abstract class OciSettingsExtensionImpl @Inject constructor(
1919
repositoryHandler: RepositoryHandler,
20-
objectFactory: ObjectFactory,
2120
providerFactory: ProviderFactory,
21+
objectFactory: ObjectFactory,
2222
buildServiceRegistry: BuildServiceRegistry,
2323
) : OciSettingsExtension {
2424

0 commit comments

Comments
 (0)