weaver 2004/08/19 09:21:30
Modified: portal/src/java/org/apache/jetspeed/engine
AbstractEngine.java
Log:
Removed the manual setting of a System prop for commons logging to use Log4j. This
was very bad, as it caused anything in the contianer
to try and use the Log4jFactory, which will break Tomcat 5 in odd ways, as it no
longer has the commons logging implementation classes
in the server classpath it only has the commons logging api interfaces.
Revision Changes Path
1.9 +1 -6
jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/engine/AbstractEngine.java
Index: AbstractEngine.java
===================================================================
RCS file:
/home/cvs/jakarta-jetspeed-2/portal/src/java/org/apache/jetspeed/engine/AbstractEngine.java,v
retrieving revision 1.8
retrieving revision 1.9
diff -u -r1.8 -r1.9
--- AbstractEngine.java 18 Aug 2004 01:14:32 -0000 1.8
+++ AbstractEngine.java 19 Aug 2004 16:21:30 -0000 1.9
@@ -25,7 +25,6 @@
import org.apache.commons.configuration.Configuration;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
-import org.apache.commons.logging.impl.Log4jFactory;
import org.apache.jetspeed.JetspeedPortalContext;
import org.apache.jetspeed.PortalContext;
import org.apache.jetspeed.PortalReservedParameters;
@@ -123,10 +122,6 @@
p.setProperty(APPLICATION_ROOT_KEY, context.getApplicationRoot());
PropertyConfigurator.configure(p);
log.info("Configured log4j from " + log4jFile);
-
- // Set up Commons Logging to use the Log4J Logging
- System.getProperties().setProperty(LogFactory.class.getName(),
- Log4jFactory.class.getName());
// patch up OJB
ClassLoader ploader2 = this.getClass().getClassLoader();
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
