ribafish opened a new pull request, #15537:
URL: https://github.com/apache/grails-core/pull/15537
## Summary
### FindMainClassTask: remove upToDateWhen preventing cache hits
- Remove `outputs.upToDateWhen` closure that checked if the cache file
exists — per Gradle docs, returning false prevents both up-to-date checks
**and** build cache loading, making `@CacheableTask` ineffective
- Remove redundant manual cache file deletion in the task action — Gradle
automatically cleans task outputs before re-execution
Both checks are redundant since Gradle already tracks the `@OutputFile`
annotation and knows to re-execute when the output is missing.
### GroovyPageForkCompileTask: fix overlapping outputs
- `compileGroovyPages` and `compileWebappGroovyPages` both wrote to
`build/gsp-classes/main`, causing Gradle to detect overlapping outputs and
disable caching for both tasks
- Give `compileWebappGroovyPages` its own output directory
(`build/gsp-classes/webapp`) and update War/Jar packaging to include both
directories
- Runtime classpath is unaffected because `output.dir('gsp-classes')`
already registers the parent directory
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
