Go back to com.github.johnrengelman.shadow and upgrade ASM to 9.6 (#1114

) Fixes #1113 --------- Co-authored-by: brharrington <brharrington@gmail.com>
Netflix · Feb 3, 2024 · 030f5bd · 030f5bd
1 parent 937add9
commit 030f5bd
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 6 deletions.
diff --git a/build.gradle b/build.gradle
@@ -1,6 +1,22 @@
+// Needed until shadow plugin is updated for JDK21. For more details see
+// https://github.com/Netflix/spectator/issues/1113
+buildscript {
+  configurations.configureEach { configuration ->
+    configuration.resolutionStrategy {
+      eachDependency { DependencyResolveDetails details ->
+        if (details.requested.group == 'org.ow2.asm') {
+          details.useVersion '9.6'
+          details.because "Asm 9.6 is required for JDK 21+ support"
+        }
+      }
+    }
+  }
+}
+
 plugins {
   id 'com.github.ben-manes.versions' version '0.50.0'
   id 'com.github.spotbugs' version '5.2.5' apply false
+  id "com.github.johnrengelman.shadow" version "8.1.1" apply false
   id 'me.champeau.jmh' version '0.7.2'
   id 'com.netflix.nebula.dependency-recommender' version '12.2.0'
   id 'com.netflix.nebula.netflixoss' version '11.4.0'

diff --git a/spectator-agent/build.gradle b/spectator-agent/build.gradle
@@ -1,5 +1,5 @@
 plugins {
-  id 'io.github.goooler.shadow' version '8.1.3'
+  id "com.github.johnrengelman.shadow"
 }
 
 dependencies {

diff --git a/spectator-ext-spark/build.gradle b/spectator-ext-spark/build.gradle
@@ -1,5 +1,5 @@
 plugins {
-  id 'io.github.goooler.shadow' version '8.1.3'
+  id "com.github.johnrengelman.shadow"
 }
 
 dependencies {

diff --git a/spectator-reg-atlas/build.gradle b/spectator-reg-atlas/build.gradle
@@ -1,7 +1,7 @@
 import java.util.zip.ZipFile
 
 plugins {
-  id 'io.github.goooler.shadow' version '8.1.3'
+    id "com.github.johnrengelman.shadow"
 }
 
 dependencies {
@@ -51,7 +51,6 @@ afterEvaluate {
   publishing {
     publications {
       withType(MavenPublication) {
-        artifact(shadowJar)
         pom.withXml {
           asNode()
             .dependencies

diff --git a/spectator-web-spring/build.gradle b/spectator-web-spring/build.gradle
@@ -16,7 +16,7 @@
 import java.util.zip.ZipFile
 
 plugins {
-  id 'io.github.goooler.shadow' version '8.1.3'
+  id "com.github.johnrengelman.shadow"
 }
 
 dependencies {
@@ -65,7 +65,6 @@ afterEvaluate {
   publishing {
     publications {
       withType(MavenPublication) {
-        artifact(shadowJar)
         pom.withXml {
           asNode()
               .dependencies