svn commit: r1351655 - /openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/TempClassLoader.java

[rmannibucau]

Author: rmannibucau
Date: Tue Jun 19 11:23:15 2012
New Revision: 1351655

URL: http://svn.apache.org/viewvc?rev=1351655&view=rev
Log:
avoid to kload log4j in a temp classloader, log4j doesn't support it

Modified:
    
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/TempClassLoader.java

Modified: 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/TempClassLoader.java
URL: 
http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/TempClassLoader.java?rev=1351655&r1=1351654&r2=1351655&view=diff
==============================================================================
--- 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/TempClassLoader.java
 (original)
+++ 
openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/core/TempClassLoader.java
 Tue Jun 19 11:23:15 2012
@@ -45,8 +45,8 @@ import java.util.Set;
  */
 // Note: this class is a fork from OpenJPA
 public class TempClassLoader extends URLClassLoader {
-    // log4j is optional
-    private static final boolean SKIP_LOG4J = 
skipLib("org.apache.log4j.Logger");
+    // log4j is optional, moreover it will likely not work if not skipped and 
loaded by a temp classloader
+    private static final boolean SKIP_LOG4J = 
"true".equals(SystemInstance.get().getProperty("openejb.skip.log4j", "true")) 
|| skipLib("org.apache.log4j.Logger");
     // commons-net is only in tomee-plus
     private static final boolean SKIP_COMMONS_NET = 
skipLib("org.apache.commons.net.pop3.POP3Client");