Author: markt Date: Mon Feb 21 15:31:05 2011 New Revision: 1073019 URL: http://svn.apache.org/viewvc?rev=1073019&view=rev Log: Withdraw patch while I work on a better one
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=1073019&r1=1073018&r2=1073019&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/STATUS.txt (original) +++ tomcat/tc6.0.x/trunk/STATUS.txt Mon Feb 21 15:31:05 2011 @@ -124,20 +124,3 @@ PATCHES PROPOSED TO BACKPORT: http://svn.apache.org/viewvc?rev=1070609&view=rev +1: markt, kkolinko -1: - -* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48863 - Better logging when specifying an invalid directory for a class loader. - Based on a patch by Ralf Hauser. - http://people.apache.org/~markt/patches/2011-02-18-bug48870.patch - +1: markt - -0: kkolinko: If I understand it corretly, it will nag about missing - ${catalina.base}/lib when running with separate home and base and that - directory does not exist. I think that is too common issue to use warn(), - and will need i18n if we would use info() and maybe use a simpler - message for the common "does not exist" case. That will need a new - patch. - For the proposed patch: I am +1 to use it as is if s/log.warn/log.debug/. - While we are at it, maybe add messages for the IS_JAR, IS_GLOB branches - as well. - -1: - --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org