You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by rf...@apache.org on 2015/01/25 22:09:26 UTC

maven-wagon git commit: [WAGON-409] Wagon-webdav-jackrabbit throws Class Not Found Exception. Use org.slf4j:jcl-over-slf4j with compile scope instead of commons-logging:commons-logging Since org.apache.commons.logging.LogFactory was already within severa

Repository: maven-wagon
Updated Branches:
  refs/heads/master f4b0c9c30 -> 6543c88c0


[WAGON-409] Wagon-webdav-jackrabbit throws Class Not Found Exception.
Use org.slf4j:jcl-over-slf4j with compile scope instead of commons-logging:commons-logging
Since org.apache.commons.logging.LogFactory was already within several test-scoped jars, this issue wasn't detected by unittests


Project: http://git-wip-us.apache.org/repos/asf/maven-wagon/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven-wagon/commit/6543c88c
Tree: http://git-wip-us.apache.org/repos/asf/maven-wagon/tree/6543c88c
Diff: http://git-wip-us.apache.org/repos/asf/maven-wagon/diff/6543c88c

Branch: refs/heads/master
Commit: 6543c88c0099e22c845552bbbc9862d517b7be1b
Parents: f4b0c9c
Author: Robert Scholte <rf...@codehaus.org>
Authored: Sun Jan 25 22:08:49 2015 +0100
Committer: Robert Scholte <rf...@codehaus.org>
Committed: Sun Jan 25 22:08:49 2015 +0100

----------------------------------------------------------------------
 wagon-providers/wagon-webdav-jackrabbit/pom.xml | 23 +++++---------------
 1 file changed, 6 insertions(+), 17 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/6543c88c/wagon-providers/wagon-webdav-jackrabbit/pom.xml
----------------------------------------------------------------------
diff --git a/wagon-providers/wagon-webdav-jackrabbit/pom.xml b/wagon-providers/wagon-webdav-jackrabbit/pom.xml
index 602376d..45cd1a3 100644
--- a/wagon-providers/wagon-webdav-jackrabbit/pom.xml
+++ b/wagon-providers/wagon-webdav-jackrabbit/pom.xml
@@ -67,6 +67,10 @@ under the License.
       <groupId>org.slf4j</groupId>
       <artifactId>slf4j-api</artifactId>
     </dependency>
+    <dependency> <!-- required for commons-httpclient:commons-httpclient -->
+      <groupId>org.slf4j</groupId>
+      <artifactId>jcl-over-slf4j</artifactId>
+    </dependency>
 
     <dependency>
       <groupId>commons-lang</groupId>
@@ -84,6 +88,7 @@ under the License.
       <!--
         The version of commons-logging that comes transitively has class loader
         problems. So we exclude it here and add a newer version below.
+        Is replaced by org.slf4j:jcl-over-slf4j
       -->
       <exclusions>
         <exclusion>
@@ -92,16 +97,6 @@ under the License.
         </exclusion>
       </exclusions>
     </dependency>
-    
-    <!--
-    This dependency is here to upgrade to a version of commons-logging that is
-    newer than the one that comes transitively from commons-httpclient above.
-    -->
-    <dependency>
-       <groupId>commons-logging</groupId>
-       <artifactId>commons-logging</artifactId>
-       <version>1.1.1</version>
-    </dependency>
 
     <!-- Test dependencies -->
     <dependency>
@@ -122,13 +117,7 @@ under the License.
       <artifactId>slf4j-log4j12</artifactId>
       <scope>test</scope>
     </dependency>
-
-    <dependency>
-      <groupId>org.slf4j</groupId>
-      <artifactId>jcl-over-slf4j</artifactId>
-      <scope>test</scope>
-    </dependency>
-
+ 
     <dependency>
       <groupId>biz.aQute</groupId>
       <artifactId>bnd</artifactId>