Avoid eagerly resolving input files in ProtobufExtract (#713) (v0.9.x backport)

[YifeiZhuang]

Rely on FileCollection.getElements() (available in Gradle 5.6+) to ensure no files are accessed eagerly during configuration. This also ensures configuration cache key does not depend on them.

Fixes issue #711.

Test: Added "testProjectDependent proto extraction with configuration cache" with Gradle 8.1

backport of #713

[TinyTrebuchet]

Hi @YifeiZhuang , I was facing a similar issue, and was checking out your fix, but had trouble understanding it.

If I am not wrong, aren't these two things effectively the same?

Create a provider, so that inputFiles.files is accessed lazily during execution, when the tasks queries the fileCollection and the sources added need to be resolved.

return objectFactory.fileCollection()
        .from(providerFactory.provider { unused ->
          Set<File> files = inputFiles.files
          // use files

Use FileCollection.getElements() which is also just provides Provider for accessing inputFile.files

return objectFactory.fileCollection()
        .from(inputFiles.getElements().map { files ->
          // use files

How does the former change access inputFiles.files eagerly?