Use Action<T> instead of Closure in API (e.g. ProjectPackagingExtension)

[vlsi]

Sample:

gradle-ospackage-plugin/src/main/groovy/com/netflix/gradle/plugins/packaging/ProjectPackagingExtension.groovy

Lines 63 to 67 in c907bb2

	CopySpec from(Object sourcePath, Closure c) {
	use(CopySpecEnhancement) {
	return getDelegateCopySpec().from(sourcePath, c)
	}
	}

It causes Kotlin DSL usages look awkward:

from("${pack}/bin", closureOf<CopySpec> {
    into("${dest}/bin")
    fileMode = 0x168
})

If the method was declared with Action<CopySpec> c, then both Groovy DSL and Kotlin DSL would be ok.

    CopySpec from(Object sourcePath, Action<CopySpec> c) {
        ...
    }

See https://docs.gradle.org/current/userguide/groovy_build_script_primer.html#delegation

All new Gradle APIs declare an Action argument type rather than Closure, which makes it very easy to pick out the delegate type. Even older APIs have an Action variant in addition to the old Closure one

[chali]

Hi @vlsi thank you for the issue. I think it makes sense. Unfortunately, our priorities don't allow us to pick up this work at this moment. Would you be willing to provide a PR?