Re: ClassLoader Repository Selector inclusion

2002-12-23 Thread Jacob Kjome
And why don't we try this one last time with the changes to doIdempotentInitialization() that I recommended in my original email.  Sorry, must have been too tired last night Jake /* * Copyright (C) The Apache Software Foundation. All rights reserved. * * This software is published unde

Re: ClassLoader Repository Selector inclusion

2002-12-23 Thread Ceki Gülcü
At 23:39 22.12.2002 -0600, Jacob Kjome wrote: Yep, this seems to work great! I've tested the code you attached here and it works flawlessly, except I see one possible issue with this. Mark wanted a Log4jInit servlet which was flexible enough to allow configuration of which repository sel

Re: ClassLoader Repository Selector inclusion

2002-12-22 Thread Jacob Kjome
Whoops, forgot to change the names of Log4jCRS to ContextClassLoaderSelector inside the class.  Here is a proper copy... Jake /* * Copyright (C) The Apache Software Foundation. All rights reserved. * * This software is published under the terms of the Apache Software * License version 1.1

Re: ClassLoader Repository Selector inclusion

2002-12-22 Thread Jacob Kjome
Hi Ceki, See comments inline below... At 03:05 PM 12/13/2002 +0100, you wrote: Many thanks for these explanations Jacob. Here are some comments. I have included a modified version of Log4jCRS. At 00:10 13.12.2002 -0600, Jacob Kjome wrote: Here is how it works... Each application, at startup, co

RE: ClassLoader Repository Selector inclusion

2002-12-13 Thread Mark Womack
Hi all, > I have a couple of minor code comments/questions: > > >(the latter being the best choice). Before any call to any > configure() > >method, the following gets run... > > > >Log4jCRS crs = new Log4jCRS(); > >crs.initLoggerRepository(); > > Why doesn't the log4jCRS constructor init the

Re: ClassLoader Repository Selector inclusion

2002-12-13 Thread Jacob Kjome
Hi Ceki and Yoav, I'll take a look at Ceki's mods and look into Yoav's suggestions either today or later this weekend. thanks for the comments. I'm very open to any changes that make improvements to the code and to the usability of the class. Jake

RE: ClassLoader Repository Selector inclusion

2002-12-13 Thread Jacob Kjome
this information should be provided in an accompanying readme or incorporated into the javadoc for the class. -Mark > -Original Message- > From: Jacob Kjome [mailto:[EMAIL PROTECTED]] > Sent: Thursday, December 12, 2002 10:10 PM > To: Log4J Developers List > Subject: Re: Cl

Re: ClassLoader Repository Selector inclusion

2002-12-13 Thread Ceki Gülcü
Many thanks for these explanations Jacob. Here are some comments. I have included a modified version of Log4jCRS. At 00:10 13.12.2002 -0600, Jacob Kjome wrote: In fact, this version of Log4jCRS is *not* "part of the Container". It is driven by applications individually. Tomcat needs to know

RE: ClassLoader Repository Selector inclusion

2002-12-13 Thread Shapira, Yoav
Howdy, >I thought I should add a note about Log4jCRS so people can understand how >it is differently implemented than the way Ceki describes it in his doc ( Thank you for the explanation. I agree with Senor Womack that this should be documented in the log4jCRS distribution. I have a couple of

RE: ClassLoader Repository Selector inclusion

2002-12-12 Thread mwomack
gt; Subject: Re: ClassLoader Repository Selector inclusion > > > > I thought I should add a note about Log4jCRS so people can understand how > it is differently implemented than the way Ceki describes it in his doc > http://www.qos.ch/logging/sc.html ). > > > >

Re: ClassLoader Repository Selector inclusion

2002-12-12 Thread Jacob Kjome
I thought I should add a note about Log4jCRS so people can understand how it is differently implemented than the way Ceki describes it in his doc ( http://www.qos.ch/logging/sc.html ). The Servlet Container will set the repository selector to a CRS instance when it starts up. This is as simp