Centralize dependency management of Commons Compress

Closes gh-39368
spring-projects · Nov 4, 2024 · a3168e3 · a3168e3
1 parent 90b920a
commit a3168e3
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 31 deletions.
diff --git a/spring-boot-project/spring-boot-parent/build.gradle b/spring-boot-project/spring-boot-parent/build.gradle
@@ -33,6 +33,13 @@ bom {
 			]
 		}
 	}
+	library("Commons Compress", "1.21") {
+		group("org.apache.commons") {
+			modules = [
+				"commons-compress"
+			]
+		}
+	}
 	library("Commons FileUpload", "1.5") {
 		group("commons-fileupload") {
 			modules = [

diff --git a/spring-boot-project/spring-boot-tools/spring-boot-buildpack-platform/build.gradle b/spring-boot-project/spring-boot-tools/spring-boot-buildpack-platform/build.gradle
@@ -19,17 +19,6 @@ configurations.all {
 			if (dependency.requested.group.equals("org.springframework")) {
 				dependency.useVersion("$springFramework60xVersion")
 			}
-			// We manage the version of commons-compress here rather than
-			// in spring-boot-parent to minimize conflicts with Testcontainers
-			if (dependency.requested.group.equals("org.apache.commons")
-					&& dependency.requested.name.equals("commons-compress")) {
-				dependency.useVersion("$commonsCompressVersion")
-			}
-			// Downgrade Testcontainers for compatibility with the managed
-			// version of Commons Compress.
-			if (dependency.requested.group.equals("org.testcontainers")) {
-				dependency.useVersion("1.19.3")
-			}
 		}
 	}
 }
@@ -38,7 +27,7 @@ dependencies {
 	api("com.fasterxml.jackson.core:jackson-databind")
 	api("com.fasterxml.jackson.module:jackson-module-parameter-names")
 	api("net.java.dev.jna:jna-platform")
-	api("org.apache.commons:commons-compress:$commonsCompressVersion")
+	api("org.apache.commons:commons-compress")
 	api("org.apache.httpcomponents.client5:httpclient5")
 	api("org.springframework:spring-core")
 	api("org.tomlj:tomlj:1.0.0")

diff --git a/spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle b/spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/build.gradle
@@ -59,17 +59,6 @@ configurations {
 				if (dependency.requested.group.equals("org.springframework")) {
 					dependency.useVersion("$springFramework60xVersion")
 				}
-				// We manage the version of commons-compress here rather than
-				// in spring-boot-parent to minimize conflicts with Testcontainers
-				if (dependency.requested.group.equals("org.apache.commons")
-						&& dependency.requested.name.equals("commons-compress")) {
-					dependency.useVersion("$commonsCompressVersion")
-				}
-				// Downgrade Testcontainers for compatibility with the managed
-				// version of Commons Compress.
-				if (dependency.requested.group.equals("org.testcontainers")) {
-					dependency.useVersion("1.19.3")
-				}
 			}
 		}
 	}
@@ -93,7 +82,7 @@ dependencies {
 	implementation(project(":spring-boot-project:spring-boot-tools:spring-boot-buildpack-platform"))
 	implementation(project(":spring-boot-project:spring-boot-tools:spring-boot-loader-tools"))
 	implementation("io.spring.gradle:dependency-management-plugin")
-	implementation("org.apache.commons:commons-compress:$commonsCompressVersion")
+	implementation("org.apache.commons:commons-compress")
 	implementation("org.springframework:spring-core")
 
 	optional("org.graalvm.buildtools:native-gradle-plugin")

diff --git a/spring-boot-project/spring-boot-tools/spring-boot-loader-tools/build.gradle b/spring-boot-project/spring-boot-tools/spring-boot-loader-tools/build.gradle
@@ -28,19 +28,13 @@ configurations {
 				if (dependency.requested.group.equals("org.springframework")) {
 					dependency.useVersion("$springFramework60xVersion")
 				}
-				// We manage the version of commons-compress here rather than
-				// in spring-boot-parent to minimize conflicts with Testcontainers
-				if (dependency.requested.group.equals("org.apache.commons")
-						&& dependency.requested.name.equals("commons-compress")) {
-					dependency.useVersion("$commonsCompressVersion")
-				}
 			}
 		}
 	}
 }
 
 dependencies {
-	api("org.apache.commons:commons-compress:$commonsCompressVersion")
+	api("org.apache.commons:commons-compress")
 	api("org.springframework:spring-core")
 
 	compileOnly("ch.qos.logback:logback-classic")