codeconsole commented on PR #16327: URL: https://github.com/apache/grails-core/pull/16327#issuecomment-5609755270
@matrei Agreed — matching the exact declared output preserves the old wiring without inheriting every compilation dependency. Implemented in 2d7fdf019a. The lookup runs only when the configured script carries no producer metadata. It matches the file against direct compile dependencies, excludes the generator itself, and adds only matching producers. Provider-wired scripts keep their existing handling. The tests now assert success for the plain-file form in natural and explicitly reordered invocations. Both wiring forms cover clean builds, unchanged builds, changed script content, configuration-cache reuse after cleaning, and an unrelated prerequisite that declares a different output file and depends on script preparation. I also included the merged #16325 cycle guard. Three plain-file cases failed before the fix and now pass. The guide recommends provider wiring while documenting that the plain-file-plus-dependsOn form remains supported when the producer declares the exact script file as an output. The new section is numbered 58 to follow the guide additions now on 8.0.x. Verification: all 272 plugin tests passed, including 17 compiler-configuration cases; plugin style and dependency validation passed. The full root clean/violations/aggregate-test check passed (18,404 tests reported, zero failures/errors, 447 skipped), all four violation reports are clean, and the guide builds successfully. -- 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]
