svn commit: r830239 - in /cxf/trunk: common/common/src/main/java/org/apache/cxf/common/logging/ parent/ rt/javascript/ rt/transports/http-jetty/ systests/databinding/src/test/java/org/apache/cxf/systest/aegis/

[dkulp]

Author: dkulp
Date: Tue Oct 27 16:38:17 2009
New Revision: 830239

URL: http://svn.apache.org/viewvc?rev=830239&view=rev
Log:
[CXF-2501] Detect where slf4j is logging and try and do the same
Also fix version of slf4j-api jar being picked up

Modified:
    
cxf/trunk/common/common/src/main/java/org/apache/cxf/common/logging/LogUtils.java
    cxf/trunk/parent/pom.xml
    cxf/trunk/rt/javascript/pom.xml
    cxf/trunk/rt/transports/http-jetty/pom.xml
    
cxf/trunk/systests/databinding/src/test/java/org/apache/cxf/systest/aegis/AegisClientServerTest.java

Modified: 
cxf/trunk/common/common/src/main/java/org/apache/cxf/common/logging/LogUtils.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/common/common/src/main/java/org/apache/cxf/common/logging/LogUtils.java?rev=830239&r1=830238&r2=830239&view=diff
==============================================================================
--- 
cxf/trunk/common/common/src/main/java/org/apache/cxf/common/logging/LogUtils.java
 (original)
+++ 
cxf/trunk/common/common/src/main/java/org/apache/cxf/common/logging/LogUtils.java
 Tue Oct 27 16:38:17 2009
@@ -76,6 +76,20 @@
                     cname = din.readLine();
                 }
             }
+            if (StringUtils.isEmpty(cname)) {
+                Class.forName("org/slf4j/impl/StaticLoggerBinder");
+                Class<?> cls = Class.forName("org.slf4j.LoggerFactory");
+                Class<?> fcls = 
cls.getMethod("getILoggerFactory").invoke(null).getClass();
+                if (fcls.getName().contains("Log4j")) {
+                    cname = "org.apache.cxf.common.logging.Log4jLogger";
+                } else if (fcls.getName().contains("JCL")) {
+                    cls = 
Class.forName("org.apache.commons.logging.LogFactory");
+                    fcls = cls.getMethod("getFactory").invoke(null).getClass();
+                    if (fcls.getName().contains("Log4j")) {
+                        cname = "org.apache.cxf.common.logging.Log4jLogger";
+                    }
+                }
+            }
             if (!StringUtils.isEmpty(cname)) {
                 loggerClass = Class.forName(cname, true,
                                             
Thread.currentThread().getContextClassLoader());

Modified: cxf/trunk/parent/pom.xml
URL: 
http://svn.apache.org/viewvc/cxf/trunk/parent/pom.xml?rev=830239&r1=830238&r2=830239&view=diff
==============================================================================
--- cxf/trunk/parent/pom.xml (original)
+++ cxf/trunk/parent/pom.xml Tue Oct 27 16:38:17 2009
@@ -563,6 +563,11 @@
                 <version>1.5.8</version>
             </dependency>
             <dependency>
+                <groupId>org.slf4j</groupId>
+                <artifactId>slf4j-api</artifactId>
+                <version>1.5.8</version>
+            </dependency>
+            <dependency>
                 <groupId>org.mortbay.jetty</groupId>
                 <artifactId>jetty</artifactId>
                 <version>${jetty.version}</version>

Modified: cxf/trunk/rt/javascript/pom.xml
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/javascript/pom.xml?rev=830239&r1=830238&r2=830239&view=diff
==============================================================================
--- cxf/trunk/rt/javascript/pom.xml (original)
+++ cxf/trunk/rt/javascript/pom.xml Tue Oct 27 16:38:17 2009
@@ -98,6 +98,11 @@
             <artifactId>slf4j-jdk14</artifactId>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>org.slf4j</groupId>
+            <artifactId>slf4j-api</artifactId>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
     <build>

Modified: cxf/trunk/rt/transports/http-jetty/pom.xml
URL: 
http://svn.apache.org/viewvc/cxf/trunk/rt/transports/http-jetty/pom.xml?rev=830239&r1=830238&r2=830239&view=diff
==============================================================================
--- cxf/trunk/rt/transports/http-jetty/pom.xml (original)
+++ cxf/trunk/rt/transports/http-jetty/pom.xml Tue Oct 27 16:38:17 2009
@@ -92,6 +92,11 @@
             <artifactId>slf4j-jdk14</artifactId>
             <scope>runtime</scope>
         </dependency>
+        <dependency>
+            <groupId>org.slf4j</groupId>
+            <artifactId>slf4j-api</artifactId>
+            <scope>runtime</scope>
+        </dependency>
 
         <dependency>
             <groupId>org.apache.geronimo.specs</groupId>

Modified: 
cxf/trunk/systests/databinding/src/test/java/org/apache/cxf/systest/aegis/AegisClientServerTest.java
URL: 
http://svn.apache.org/viewvc/cxf/trunk/systests/databinding/src/test/java/org/apache/cxf/systest/aegis/AegisClientServerTest.java?rev=830239&r1=830238&r2=830239&view=diff
==============================================================================
--- 
cxf/trunk/systests/databinding/src/test/java/org/apache/cxf/systest/aegis/AegisClientServerTest.java
 (original)
+++ 
cxf/trunk/systests/databinding/src/test/java/org/apache/cxf/systest/aegis/AegisClientServerTest.java
 Tue Oct 27 16:38:17 2009
@@ -47,7 +47,7 @@
     
     @BeforeClass
     public static void startServers() throws Exception {
-        assertTrue("server did not launch correctly", 
launchServer(AegisServer.class, true));
+        assertTrue("server did not launch correctly", 
launchServer(AegisServer.class));
     }
     
     @Test