tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Mark Thomas <ma...@apache.org>
Subject Re: svn commit: r690944 - /tomcat/current/tc5.5.x/STATUS.txt
Date Tue, 02 Sep 2008 18:25:49 GMT
rjung@apache.org wrote:
> Author: rjung
> Date: Mon Sep  1 04:50:48 2008
> New Revision: 690944
> 
> URL: http://svn.apache.org/viewvc?rev=690944&view=rev
> Log:
> Vote and comment.
> 
> Modified:
>     tomcat/current/tc5.5.x/STATUS.txt
> 
> Modified: tomcat/current/tc5.5.x/STATUS.txt
> URL: http://svn.apache.org/viewvc/tomcat/current/tc5.5.x/STATUS.txt?rev=690944&r1=690943&r2=690944&view=diff
> ==============================================================================
> --- tomcat/current/tc5.5.x/STATUS.txt (original)
> +++ tomcat/current/tc5.5.x/STATUS.txt Mon Sep  1 04:50:48 2008
> @@ -34,6 +34,10 @@
>                point. You could use System.out but that is far worse than the
>                current proposal given that this will occur once per context with
>                a default configuration and this is not an error condition.
> +      rjung - Is this happening when trying to read a per context logging.properties?
> +              If so: then why don't we already have the global logging setup and can
use it?
> +              I can see, that the code in question is above the general logging setup
code,
> +              but I would expect readConfiguration() being called for the system class
loader first?

I didn't look at this much but by the time this is called for the context
the global logging has not completed configuration. If you try and get a
logger you get a stack overflow. Careful coding may be able to avoid this
but I didn't spend much time looking.

I'd also note we just applied this patch as is to TC6.

Mark


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


Mime
View raw message