You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by cl...@apache.org on 2022/07/22 17:51:59 UTC

[activemq-artemis] 05/42: add commons-logging -> SLF4J bridge and exclude commons-logging, suppresses unconfigured log spam from beanutils etc

This is an automated email from the ASF dual-hosted git repository.

clebertsuconic pushed a commit to branch new-logging
in repository https://gitbox.apache.org/repos/asf/activemq-artemis.git

commit a120f1b2ff9eebb7eca9753d5a1d883552645a5d
Author: Robbie Gemmell <ro...@apache.org>
AuthorDate: Mon Jul 11 16:16:43 2022 +0100

    add commons-logging -> SLF4J bridge and exclude commons-logging, suppresses unconfigured log spam from beanutils etc
---
 artemis-commons/pom.xml    |  5 +++++
 artemis-jdbc-store/pom.xml |  5 +++++
 artemis-rest/pom.xml       |  5 +++++
 artemis-server/pom.xml     |  5 +++++
 pom.xml                    | 42 ++++++++++++++++++++++++++++++++----------
 5 files changed, 52 insertions(+), 10 deletions(-)

diff --git a/artemis-commons/pom.xml b/artemis-commons/pom.xml
index 5aaa2c2294..d96f8c2137 100644
--- a/artemis-commons/pom.xml
+++ b/artemis-commons/pom.xml
@@ -92,6 +92,11 @@
          <groupId>commons-beanutils</groupId>
          <artifactId>commons-beanutils</artifactId>
       </dependency>
+      <dependency>
+         <!-- Needed to replace commons-logging for commons-beanutils -->
+         <groupId>org.slf4j</groupId>
+         <artifactId>jcl-over-slf4j</artifactId>
+      </dependency>
       <dependency>
          <groupId>junit</groupId>
          <artifactId>junit</artifactId>
diff --git a/artemis-jdbc-store/pom.xml b/artemis-jdbc-store/pom.xml
index ce7c70a5c2..ee4f221609 100644
--- a/artemis-jdbc-store/pom.xml
+++ b/artemis-jdbc-store/pom.xml
@@ -64,6 +64,11 @@
          <groupId>org.apache.commons</groupId>
          <artifactId>commons-dbcp2</artifactId>
       </dependency>
+      <dependency>
+         <!-- Needed to replace commons-logging for commons-dbcp2 -->
+         <groupId>org.slf4j</groupId>
+         <artifactId>jcl-over-slf4j</artifactId>
+      </dependency>
 
       <!-- Database driver support -->
       <dependency>
diff --git a/artemis-rest/pom.xml b/artemis-rest/pom.xml
index 1c33dcef97..2ce86c5136 100644
--- a/artemis-rest/pom.xml
+++ b/artemis-rest/pom.xml
@@ -149,6 +149,11 @@
          <groupId>org.apache.httpcomponents</groupId>
          <artifactId>httpclient</artifactId>
       </dependency>
+      <dependency>
+         <!-- Needed to replace commons-logging for httpclient -->
+         <groupId>org.slf4j</groupId>
+         <artifactId>jcl-over-slf4j</artifactId>
+      </dependency>
       <dependency>
          <groupId>commons-codec</groupId>
          <artifactId>commons-codec</artifactId>
diff --git a/artemis-server/pom.xml b/artemis-server/pom.xml
index 6a5b477d5e..f69a8e3c06 100644
--- a/artemis-server/pom.xml
+++ b/artemis-server/pom.xml
@@ -158,6 +158,11 @@
          <groupId>commons-beanutils</groupId>
          <artifactId>commons-beanutils</artifactId>
       </dependency>
+      <dependency>
+         <!-- Needed to replace commons-logging for commons-beanutils -->
+         <groupId>org.slf4j</groupId>
+         <artifactId>jcl-over-slf4j</artifactId>
+      </dependency>
       <dependency>
          <groupId>org.apache.commons</groupId>
          <artifactId>commons-configuration2</artifactId>
diff --git a/pom.xml b/pom.xml
index fb9db8a07c..89949a466c 100644
--- a/pom.xml
+++ b/pom.xml
@@ -498,10 +498,16 @@
             <!-- License: Apache 2.0 -->
          </dependency>
          <dependency>
-             <groupId>org.apache.commons</groupId>
-             <artifactId>commons-configuration2</artifactId>
-             <version>${commons.config.version}</version>
-             <!-- License: Apache 2.0 -->
+            <groupId>org.apache.commons</groupId>
+            <artifactId>commons-configuration2</artifactId>
+            <version>${commons.config.version}</version>
+            <!-- License: Apache 2.0 -->
+            <exclusions>
+               <exclusion>
+                  <groupId>commons-logging</groupId>
+                  <artifactId>commons-logging</artifactId>
+               </exclusion>
+            </exclusions>
          </dependency>
          <dependency>
              <groupId>org.apache.commons</groupId>
@@ -789,14 +795,18 @@
             <artifactId>commons-beanutils</artifactId>
             <version>${commons.beanutils.version}</version>
             <!-- License: Apache 2.0 -->
+            <exclusions>
+               <exclusion>
+                  <groupId>commons-logging</groupId>
+                  <artifactId>commons-logging</artifactId>
+               </exclusion>
+            </exclusions>
          </dependency>
-
-         <!-- needed by commons-beanutils-->
+         <!-- needed to replace commons-logging for commons-beanutils etc -->
          <dependency>
-            <groupId>commons-logging</groupId>
-            <artifactId>commons-logging</artifactId>
-            <version>1.2</version>
-            <!-- License: Apache 2.0 -->
+           <groupId>org.slf4j</groupId>
+           <artifactId>jcl-over-slf4j</artifactId>
+           <version>${slf4j.version}</version>
          </dependency>
 
          <!-- needed by artemis-selector -->
@@ -822,6 +832,12 @@
             <artifactId>commons-dbcp2</artifactId>
             <version>${commons.dbcp2.version}</version>
             <!-- license Apache 2 -->
+            <exclusions>
+               <exclusion>
+                  <groupId>commons-logging</groupId>
+                  <artifactId>commons-logging</artifactId>
+               </exclusion>
+            </exclusions>
          </dependency>
          <!-- used by commons-dbcp2 -->
          <dependency>
@@ -1070,6 +1086,12 @@
             <groupId>org.apache.httpcomponents</groupId>
             <artifactId>httpclient</artifactId>
             <version>${apache.httpclient.version}</version>
+            <exclusions>
+               <exclusion>
+                  <groupId>commons-logging</groupId>
+                  <artifactId>commons-logging</artifactId>
+               </exclusion>
+            </exclusions>
          </dependency>
 
       </dependencies>