As I mentioned in my private email to you, it may not always be practical to rely on the list of "noTldJars" configured in catalina.properties if you bundle Tomcat with your own product. That's why I added the setProperty method on CatalinaProperties, in order to be able to override the value in catalina.properties. This allows the list of "noTldJars" to be determined more accurately at runtime. The code that is embedding Tomcat would call CatalinaProperties.setProperty() *before* starting Tomcat.
And is exactly what I wanted to avoid. Again, CatalinaProperties is for configuring the bootstrap, not anything else (otherwise, stuff which belongs to server.xml will end up here, and it will end up being a huge mess).
I agree w/ that.
I leave the judgement up to the community.
I'll have to reiterate my -1.
OK, I'll revert my patch (and leave it as a private extension), until we have found a better solution that everybody agrees on.
Jan
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]