chore: Remove instrumentation afterEvaluate

# Conflicts:
#	dd-java-agent/instrumentation/build.gradle

Author: bric3

dd-java-agent/instrumentation/build.gradle

@@ -10,42 +10,56 @@ tasks.register("latestDepTest", Test)
 
 Project parent_project = project
 subprojects { Project subProj ->
-  apply plugin: 'instrument'
-  apply plugin: 'muzzle'
-
-  configurations {
-    instrumentPluginClasspath {
-      visible = false
-      canBeConsumed = false
-      canBeResolved = true
+  subProj.pluginManager.withPlugin("instrument") {
+    subProj.extensions.configure(InstrumentExtension) {
+      it.plugins.addAll(
+        'datadog.trace.agent.tooling.muzzle.MuzzleGradlePlugin',
+        'datadog.trace.agent.tooling.bytebuddy.NewTaskForGradlePlugin',
+        'datadog.trace.agent.tooling.bytebuddy.reqctx.RewriteRequestContextAdvicePlugin',
+      )
     }
-  }
 
-  instrument.plugins = [
-    'datadog.trace.agent.tooling.muzzle.MuzzleGradlePlugin',
-    'datadog.trace.agent.tooling.bytebuddy.NewTaskForGradlePlugin',
-    'datadog.trace.agent.tooling.bytebuddy.reqctx.RewriteRequestContextAdvicePlugin',
-  ]
+    subProj.configurations.register("instrumentPluginClasspath") {
+      it.visible = false
+      it.canBeConsumed = false
+      it.canBeResolved = true
 
-  subProj.tasks.withType(Javadoc).configureEach { enabled = false }
+      it.dependencies.add(subProj.dependencies.project(path: ':dd-java-agent:agent-tooling', configuration: 'instrumentPluginClasspath'))
+    }
 
-  subProj.afterEvaluate {
-    if (!plugins.hasPlugin("java")) {
-      return
+  }
+
+  subProj.pluginManager.withPlugin("java") {
+    subProj.pluginManager.withPlugin("muzzle") {
+      subProj.configurations.matching { it.name == 'muzzleBootstrap' }.configureEach {
+        exclude group: 'org.snakeyaml', module: 'snakeyaml-engine' // we vendor this in the agent jar
+      }
     }
 
     // Configures base dependencies for additional sourceSet
-    configurations
+    subProj.configurations
       .matching { it.name.matches("${SourceSet.MAIN_SOURCE_SET_NAME}_java\\d+${JavaPlugin.IMPLEMENTATION_CONFIGURATION_NAME.capitalize()}") }
       .configureEach {
         it.dependencies.add(project.dependencyFactory.create(project(':dd-trace-api')))
         it.dependencies.add(project.dependencyFactory.create(project(':dd-java-agent:agent-tooling')))
         it.dependencies.addLater(libs.bytebuddy)
       }
 
-    configurations.named('muzzleBootstrap') {
-      exclude group: 'org.snakeyaml', module: 'snakeyaml-engine' // we vendor this in the agent jar
+    subProj.tasks.withType(Javadoc).configureEach { enabled = false }
+
+    if (subProj.hasProperty('minJavaVersionForTests') && subProj.findProperty('minJavaVersionForTests') != JavaVersion.VERSION_1_8) {
+      def version = JavaVersion.toVersion(subProj.findProperty('minJavaVersionForTests'))
+      def mainJavaConfigName = "main_name${version.majorVersion}Implementation"
+
+      configurations
+        .matching {it.name == mainJavaConfigName }
+        .configureEach {
+          it.dependencies.add(subProj.dependencies.project(path: ':dd-trace-api'))
+          it.dependencies.add(subProj.dependencies.project(path: ':dd-java-agent:agent-tooling'))
+          it.dependencies.addLater(libs.bytebuddy)
+      }
     }
+
     dependencies {
       // Apply common dependencies for instrumentation.
       implementation project(':dd-trace-api')
@@ -65,26 +79,24 @@ subprojects { Project subProj ->
       testImplementation project(':dd-java-agent:instrumentation-testing')
       testAnnotationProcessor libs.autoservice.processor
       testCompileOnly libs.autoservice.annotation
-
-      instrumentPluginClasspath project(path: ':dd-java-agent:agent-tooling', configuration: 'instrumentPluginClasspath')
     }
 
     subProj.tasks.withType(Test).configureEach { subTask ->
       if (subTask.name in ['latestDepTest', 'latestDepForkedTest']) {
         subTask.jvmArgs '-Dtest.dd.latestDepTest=true'
       }
     }
-  }
 
-  def path = subProj.getPath()
-  subProj.plugins.withId("java") {
-    if (!path.equals(':dd-java-agent:instrumentation:vertx:vertx-redis-client-3.9:stubs')) {
-      // don't include the redis RequestImpl stub
+    if (subProj.path != ':dd-java-agent:instrumentation:vertx:vertx-redis-client-3.9:stubs') {
+      // don't include the redis RequestImpl stubs
       parent_project.dependencies {
-        implementation project(path)
+        addProvider("implementation", providers.provider { project(subProj.path) })
       }
     }
   }
+
+  subProj.apply plugin: 'instrument'
+  subProj.apply plugin: 'muzzle'
 }
 
 dependencies {