Author: kkolinko
Date: Thu Nov 12 15:01:56 2009
New Revision: 835404

URL: http://svn.apache.org/viewvc?rev=835404&view=rev
Log:
vote and proposal

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=835404&r1=835403&r2=835404&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Thu Nov 12 15:01:56 2009
@@ -154,6 +154,23 @@
   integration
   
http://people.apache.org/~markt/patches/2009-11-06-log-config-per-instance.patch
   +1: markt, funkman
+  +1: kkolinko (good, though I propose a slightly corrected version of it 
below)
+  -1:
+
+  kkolinko:
+   I updated the above Mark's patch with the following changes
+   (in catalina.bat/sh):
+    - Added an explicit check that CATALINA_BASE != CATALINA_HOME.
+      Otherwise the [ -r "$CATALINA_BASE/bin/tomcat-juli.jar" ] check will 
succeed
+      when those are equal.
+    - Moved the code that adds ";" or ":" to non-empty CLASSPATH. That is to 
avoid
+      the gap between this preparationary step and the one that actually adds 
jars
+      to the CLASSPATH.
+   Warning: I have not tried to run the patched catalina.sh yet. I hope it 
works.
+  
http://people.apache.org/~kkolinko/patches/2009-11-12_log-config-per-instance.patch
+  +1: kkolinko
+  -1:
+
 
 * Make FileHandler.java extensible
   
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/juli/FileHandler.java?r1=666232&r2=709018&pathrev=793882&view=patch



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to