You are viewing a plain text version of this content. The canonical link for it is here.

Posted to scm@geronimo.apache.org by jd...@apache.org on 2006/09/04 14:09:57 UTC

svn commit: r440042 - /geronimo/genesis/trunk/plugins/plugin-support/src/main/java/org/apache/geronimo/genesis/MojoSupport.java

Author: jdillon
Date: Mon Sep  4 05:09:57 2006
New Revision: 440042

URL: http://svn.apache.org/viewvc?view=rev&rev=440042
Log:
Use commons-logging.properties to set the Log impl, seems to be better behaved

Modified:
    geronimo/genesis/trunk/plugins/plugin-support/src/main/java/org/apache/geronimo/genesis/MojoSupport.java

Modified: geronimo/genesis/trunk/plugins/plugin-support/src/main/java/org/apache/geronimo/genesis/MojoSupport.java
URL: http://svn.apache.org/viewvc/geronimo/genesis/trunk/plugins/plugin-support/src/main/java/org/apache/geronimo/genesis/MojoSupport.java?view=diff&rev=440042&r1=440041&r2=440042
==============================================================================
--- geronimo/genesis/trunk/plugins/plugin-support/src/main/java/org/apache/geronimo/genesis/MojoSupport.java (original)
+++ geronimo/genesis/trunk/plugins/plugin-support/src/main/java/org/apache/geronimo/genesis/MojoSupport.java Mon Sep  4 05:09:57 2006
@@ -52,7 +52,14 @@
         //
 
         System.setProperty("org.apache.commons.logging.LogFactory", "org.apache.commons.logging.impl.LogFactoryImpl");
-        System.setProperty("org.apache.commons.logging.Log", "org.apache.geronimo.genesis.MavenPluginLog");
+
+        //
+        // NOTE: org.apache.commons.logging.Log is set in commons-logging.properties.  Hard-coding this here
+        //       causes some other Maven plugins to have problems (like the site plugin when it runs checkstyle).
+        //       Not sure that this will always get picked up though... :-(
+        //
+        // System.setProperty("org.apache.commons.logging.Log", "org.apache.geronimo.genesis.MavenPluginLog");
+
         System.setProperty("geronimo.bootstrap.logging.enabled", "false");
     }