Upgrade to Spring Loaded 1.2.5 and allow agent config. Fixes #9309, #9201, #8817 and #9288

Author: graemerocher

build.gradle

@@ -44,7 +44,7 @@ ext {
     reactorVersion = '2.0.6.RELEASE'
 
     spockVersion = '1.0-groovy-2.4'
-    springLoadedVersion = "1.2.4.RELEASE"
+    springLoadedVersion = "1.2.5.RELEASE"
     springBootVersion = "1.2.7.RELEASE"
     springLoadedCommonOptions = "-Xverify:none -Dspringloaded.synchronize=true -Djdk.reflect.allowGetCallerClass=true"
     springVersion = "4.1.8.RELEASE"

grails-gradle-plugin/src/main/groovy/org/grails/gradle/plugin/agent/AgentTasksEnhancer.groovy

@@ -5,6 +5,7 @@ import org.gradle.api.Action
 import org.gradle.api.Project
 import org.gradle.api.Task
 import org.gradle.api.tasks.JavaExec
+import org.grails.gradle.plugin.core.GrailsExtension
 
 /**
  * Sets up the reloading agent based on the agent configuration after the project has been configured
@@ -31,8 +32,30 @@ class AgentTasksEnhancer implements Action<Project> {
     }
 
     private void addAgent(Project project, JavaExec exec, File agent) {
-        exec.jvmArgs "-javaagent:$agent.absolutePath"
-        exec.jvmArgs "-Xverify:none"
-        exec.jvmArgs "-Dspringloaded=inclusions=grails.plugins..*"
+
+        GrailsExtension.Agent agentConfig = project.extensions.findByType(GrailsExtension)?.agent ?: new GrailsExtension.Agent()
+
+        exec.jvmArgs "-javaagent:${agentConfig.path?.absolutePath ?: agent.absolutePath}"
+
+        for(arg in agentConfig.jvmArgs) {
+            exec.jvmArgs arg
+        }
+        for(entry in agentConfig.systemProperties) {
+            exec.systemProperty(entry.key, entry.value)
+        }
+
+        Map<String, String> agentArgs= [
+                inclusions: agentConfig.inclusions,
+                synchronize: String.valueOf( agentConfig.synchronize ),
+                allowSplitPackages: String.valueOf( agentConfig.allowSplitPackages ),
+                cacheDir: agentConfig.cacheDir ? project.mkdir(agentConfig.cacheDir) : project.mkdir("build/springloaded")
+        ]
+        if(agentConfig.logging != null) {
+            agentArgs.put("logging", String.valueOf(agentConfig.logging))
+        }
+        if(agentConfig.exclusions) {
+            agentArgs.put('exclusions', agentConfig.exclusions)
+        }
+        exec.systemProperty('springloaded', agentArgs.collect { entry -> "$entry.key=$entry.value"}.join(';'))
     }
 }

grails-gradle-plugin/src/main/groovy/org/grails/gradle/plugin/core/GrailsExtension.groovy

@@ -1,6 +1,7 @@
 package org.grails.gradle.plugin.core
 
 import org.apache.tools.ant.taskdefs.condition.Os
+import org.gradle.util.ConfigureUtil
 
 /**
  * A extension to the Gradle plugin to configure Grails settings
@@ -19,4 +20,31 @@ class GrailsExtension {
      * Whether assets should be packaged in META-INF/assets for plugins
      */
     boolean packageAssets = true
+
+    /**
+     * Configure the reloading agent
+     */
+    Agent agent = new Agent()
+
+    /**
+     * Configure the reloading agent
+     */
+    Agent agent(Closure configurer) {
+        ConfigureUtil.configure(configurer, agent)
+    }
+    /**
+     * Configuration for the reloading agent
+     */
+    static class Agent {
+        File path
+        String inclusions = "grails.plugins..*"
+        String exclusions
+        Boolean logging
+        boolean synchronize = true
+        boolean allowSplitPackages = true
+        File cacheDir = new File("build/springloaded")
+
+        Map<String, String> systemProperties = ['jdk.reflect.allowGetCallerClass': 'true']
+        List<String> jvmArgs = ['-Xverify:none']
+    }
 }