You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2008/09/03 20:54:22 UTC

svn commit: r691730 - /tomcat/tc6.0.x/trunk/STATUS.txt

Author: remm
Date: Wed Sep  3 11:54:22 2008
New Revision: 691730

URL: http://svn.apache.org/viewvc?rev=691730&view=rev
Log:
- Votes.

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=691730&r1=691729&r2=691730&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Sep  3 11:54:22 2008
@@ -137,7 +137,7 @@
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45628
   http://svn.apache.org/viewvc?rev=691282&view=rev
   JARs without deps should always be fulfilled
-  +1: markt
+  +1: markt, remm
   -1: 
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45618
@@ -157,4 +157,5 @@
   because it is low risk (generics) and keeps the class
   in sync between trunk and tc6.0.x.
   +1: rjung
+   0: remm: the logging should be grouped into a single log entry, I think
   -1: 



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


Re: svn commit: r691730 - /tomcat/tc6.0.x/trunk/STATUS.txt

Posted by Filip Hanik - Dev Lists <de...@hanik.com>.
Rainer Jung wrote:
> Hi Remy,
>
> remm@apache.org schrieb:
>   
>>  * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45618
>> @@ -157,4 +157,5 @@
>>    because it is low risk (generics) and keeps the class
>>    in sync between trunk and tc6.0.x.
>>    +1: rjung
>> +   0: remm: the logging should be grouped into a single log entry, I think
>>    -1: 
>>     
>
> What about adding this
>
> http://people.apache.org/~rjung/patches/ClassLoaderLogManager_Log_AccessControlException.patch
>
> as well? I tried to squeeze the info in one line in case it is the
> expected exception, and only if we really don't understand what's going
> on, I still use two lines.
>
> Expected output (path truncated):
>
> Sep 3, 2008 9:56:22 PM org.apache.juli.ClassLoaderLogManager
> readConfiguration
> WARNING: Reading
> /.../webapps/examples/WEB-INF/classes/logging.properties is not
> permitted. See "per context logging" in the default catalina.policy file.
>
> BTW: Do you have a good idea how to get the context name? We have the
> WebappClassLoader, but I don't see easily how to get to the context. Any
> hints? It would be nice for the second case, i.e. unexpected exception.
>   
I can't remember, but at some point we did "named classloaders", did 
that go away?
it might have disappeared with the trunk removal a while back

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


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


Re: svn commit: r691730 - /tomcat/tc6.0.x/trunk/STATUS.txt

Posted by Remy Maucherat <re...@apache.org>.
On Wed, 2008-09-03 at 22:02 +0200, Rainer Jung wrote:
> Hi Remy,
> 
> What about adding this
> 
> http://people.apache.org/~rjung/patches/ClassLoaderLogManager_Log_AccessControlException.patch
> 
> as well? I tried to squeeze the info in one line in case it is the
> expected exception, and only if we really don't understand what's going
> on, I still use two lines.
> 
> Expected output (path truncated):
> 
> Sep 3, 2008 9:56:22 PM org.apache.juli.ClassLoaderLogManager
> readConfiguration
> WARNING: Reading
> /.../webapps/examples/WEB-INF/classes/logging.properties is not
> permitted. See "per context logging" in the default catalina.policy file.

Ok.

> BTW: Do you have a good idea how to get the context name? We have the
> WebappClassLoader, but I don't see easily how to get to the context. Any
> hints? It would be nice for the second case, i.e. unexpected exception.

To get some info on the CL, you could try getURLs()[0].toString(). Not
perfect but this avoids custom hacks.

Rémy



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


Re: svn commit: r691730 - /tomcat/tc6.0.x/trunk/STATUS.txt

Posted by Rainer Jung <ra...@kippdata.de>.
Hi Remy,

remm@apache.org schrieb:
>  * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45618
> @@ -157,4 +157,5 @@
>    because it is low risk (generics) and keeps the class
>    in sync between trunk and tc6.0.x.
>    +1: rjung
> +   0: remm: the logging should be grouped into a single log entry, I think
>    -1: 

What about adding this

http://people.apache.org/~rjung/patches/ClassLoaderLogManager_Log_AccessControlException.patch

as well? I tried to squeeze the info in one line in case it is the
expected exception, and only if we really don't understand what's going
on, I still use two lines.

Expected output (path truncated):

Sep 3, 2008 9:56:22 PM org.apache.juli.ClassLoaderLogManager
readConfiguration
WARNING: Reading
/.../webapps/examples/WEB-INF/classes/logging.properties is not
permitted. See "per context logging" in the default catalina.policy file.

BTW: Do you have a good idea how to get the context name? We have the
WebappClassLoader, but I don't see easily how to get to the context. Any
hints? It would be nice for the second case, i.e. unexpected exception.

Regards,

Rainer

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