Groovy 4

[codeconsole]

Remove large configurations block and set to groovy 4

[jamesfredley]

@codeconsole The only holdup on Groovy 4 for the grails-gradle-plugin is:

https://docs.gradle.org/current/userguide/compatibility.html#groovy "Gradle plugins written in Groovy must use Groovy 3.x for compatibility with Gradle and Groovy DSL build scripts."

[jamesfredley]

Groovy 4 appears to work and eliminates the Groovy 3 vs 4 conflicts nicely, I just wish there was confirmation that the statement from Gradle is no longer valid.

[matrei]

Groovy 4 appears to work and eliminates the Groovy 3 vs 4 conflicts nicely, I just wish there was confirmation that the statement from Gradle is no longer valid.

@codeconsole Have you tried to use the plugins from this project compiled with Groovy 4?

[matrei]

I find It's better to use configurations.configureEach as configurations.all will configure all configurations eagerly (even unused configurations), while the former does so only lazily upon usage. I also like to write a comment as to why, I for example exclude something, because then it's really easy for the next developer reading the file to understand.

[jamesfredley]

@puneetbehl submitted a PR with the Groovy 4 update, about 2 years ago, so this appears to have been planned for Grails 6, but Groovy 4 got bumped to Grails 7.

#139

[codeconsole]

Groovy 4 appears to work and eliminates the Groovy 3 vs 4 conflicts nicely, I just wish there was confirmation that the statement from Gradle is no longer valid.

@codeconsole Have you tried to use the plugins from this project compiled with Groovy 4?

:compileGroovyPages works, but :urlMappingsReport fails

./gradlew :urlMappingsReport

> Task :urlMappingsReport FAILED
Context failed to load: Bean definition [io.micronaut.spring.context.factory.ImportBeanRegistrarProcessor] could not be loaded: Could not initialize class io.micronaut.inject.annotation.AnnotationMetadataSupport

[jamesfredley]

I tested it against grails-database-migration and gorm-mongodb: assemble, build & groovydoc - no issues

and it allows removal of the following from buildScript:

configurations.configureEach {
        resolutionStrategy.eachDependency { DependencyResolveDetails details ->
            // This is a workaround for dependencies exposing a different version of Groovy
            // than the one used by Gradle. This causes issues with the Groovy compiler.
            if (details.requested.group == 'org.apache.groovy' || details.requested.group == 'org.codehaus.groovy') {
                details.useTarget(group: "org.codehaus.groovy", name: details.requested.name, version: GroovySystem.version)
                details.because "Use Groovy 3.0.x library shipped with Gradle"
            }
        }
    }

There are still some 7.0.0-SNAPSHOT dependencies that are referencing org.codehaus.groovy which separately need updated.

org.grails:grails-gsp:7.0.0-SNAPSHOT
org.grails:grails-web-gsp-taglib:7.0.0-SNAPSHOT
org.grails:grails-web-gsp:7.0.0-SNAPSHOT
org.grails:grails-web-jsp:7.0.0-SNAPSHOT

PR for these grails/grails-gsp#477

[jamesfredley]

The following should resolve Context failed to load: Bean definition [io.micronaut.spring.context.factory.ImportBeanRegistrarProcessor] could not be loaded: Could not initialize class io.micronaut.inject.annotation.AnnotationMetadataSupport

"io.micronaut:micronaut-inject-groovy:$micronautVersion"
"io.micronaut:micronaut-inject:$micronautVersion"