remm        2003/09/06 10:49:31

  Modified:    catalina/src/share/org/apache/catalina/loader
                        WebappClassLoader.java
  Log:
  - Modify the bundling of commons-logging to fix (hopefully) the nagging CL issues.
  - The commons-logging-api JAR will now be put in the system classloader.
    When using an alternate logging implmentation (ex: log4j) you should put the
    wrapper implementation in the same classloader or there will likely be trouble.
  - Ex: When using a Struts 1.1 webapp with log4j, there should be commons-logging.jar
    (just the log4j logger is fine as well) next to it.
  - Of course, overriding the log4j API in a webapp is still not possible. It wasn't
    before as c-logging was treated as a special case by the classloader (like JAXP).
  - This nasty case now works for me (bug 22701), as well as using log4j with
    privileged webapps (with or without SSL).
  
  Revision  Changes    Path
  1.25      +1 -2      
jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/loader/WebappClassLoader.java
  
  Index: WebappClassLoader.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/loader/WebappClassLoader.java,v
  retrieving revision 1.24
  retrieving revision 1.25
  diff -u -r1.24 -r1.25
  --- WebappClassLoader.java    2 Sep 2003 21:22:03 -0000       1.24
  +++ WebappClassLoader.java    6 Sep 2003 17:49:31 -0000       1.25
  @@ -192,7 +192,6 @@
           "javax",                                     // Java extensions
           "org.xml.sax",                               // SAX 1 & 2
           "org.w3c.dom",                               // DOM 1 & 2
  -        "org.apache.commons.logging",                // Commons logging.
           "org.apache.xerces",                         // Xerces 1 & 2
           "org.apache.xalan"                           // Xalan
       };
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to