Re: PATCH: modify config file loading to allow custom URL types

2001-11-27 Thread Jukka . Zitting
Hi all, This message is a revival of the June thread on with the same subject. Then a new configuration property (log4j.configuratorClass) was suggested as a solution to the problem with custom URL formats. I very much needed this functionality but could not find a patch that would have implement

Re: PATCH: modify config file loading to allow custom URL types

2001-07-11 Thread Ceki Gülcü
At 12:18 11.07.2001 -0400, Colin Sampaleanu wrote: >At 07:11 AM 11/07/2001 +0200, you wrote: >>At 18:23 10.07.2001 -0400, Colin Sampaleanu wrote: >>>I am of mixed feelings about this whole scheme of piggybacking a >>>configurator classname as a ref portion of a URL. Obviously there are cases >>>wh

Re: PATCH: modify config file loading to allow custom URL types

2001-07-11 Thread Colin Sampaleanu
At 07:11 AM 11/07/2001 +0200, you wrote: >At 18:23 10.07.2001 -0400, Colin Sampaleanu wrote: > >I am of mixed feelings about this whole scheme of piggybacking a > >configurator classname as a ref portion of a URL. Obviously there are cases > >when you can't in fact tell if it is a classname or a v

Re: PATCH: modify config file loading to allow custom URL types

2001-07-10 Thread Ceki Gülcü
torClass" instead? Regards, Ceki >- Original Message - >From: "Colin Sampaleanu" <[EMAIL PROTECTED]> >To: "LOG4J Developers Mailing List" <[EMAIL PROTECTED]> >Sent: Tuesday, July 10, 2001 5:38 PM >Subject: Re: PATCH: modify config fil

Re: PATCH: modify config file loading to allow custom URL types

2001-07-10 Thread Anders Kristensen
Colin Sampaleanu wrote: > > I am of mixed feelings about this whole scheme of piggybacking a > configurator classname as a ref portion of a URL. Obviously there are cases > when you can't in fact tell if it is a classname or a valid part of a custom > url... I would have just used a separate sys

Re: PATCH: modify config file loading to allow custom URL types

2001-07-10 Thread Colin Sampaleanu
u" <[EMAIL PROTECTED]> To: "LOG4J Developers Mailing List" <[EMAIL PROTECTED]> Sent: Tuesday, July 10, 2001 5:38 PM Subject: Re: PATCH: modify config file loading to allow custom URL types > Hi, > > Unfortunately I have to take back my comment about this code be

Re: PATCH: modify config file loading to allow custom URL types

2001-07-10 Thread Colin Sampaleanu
Sent: Thursday, July 05, 2001 12:54 PM Subject: Re: PATCH: modify config file loading to allow custom URL types Colin, I modified OptionConverter.java as follows: static public void selectAndConfigure(URL url, Hierarchy hierarchy) { String clazz = url.getRef(); if (clazz.ind

RE: PATCH: modify config file loading to allow custom URL types

2001-07-07 Thread Ceki Gülcü
At 19:55 06.07.2001 -0400, Colin Sampaleanu wrote: >> -Original Message- >> From: Ceki Gülcü [mailto:[EMAIL PROTECTED]] >> >- I think (but have no way to verify) your code will actually not work >> >properly on a Mac, since the file separator there is ':' from what I >know, >> >not / or \.

RE: PATCH: modify config file loading to allow custom URL types

2001-07-06 Thread Colin Sampaleanu
> -Original Message- > From: Ceki Gülcü [mailto:[EMAIL PROTECTED]] > >- I think (but have no way to verify) your code will actually not work > >properly on a Mac, since the file separator there is ':' from what I know, > >not / or \. This is why my code did a > >'System.getProperty("file.s

Re: PATCH: modify config file loading to allow custom URL types

2001-07-06 Thread Ceki Gülcü
it out. Regards, Ceki >- Original Message - >From: "Ceki Gülcü" <[EMAIL PROTECTED]> >To: "LOG4J Developers Mailing List" <[EMAIL PROTECTED]> >Sent: Thursday, July 05, 2001 12:54 PM >Subject: Re: PATCH: modify config file loading to all

Re: PATCH: modify config file loading to allow custom URL types

2001-07-06 Thread Colin Sampaleanu
PROTECTED]> To: "LOG4J Developers Mailing List" <[EMAIL PROTECTED]> Sent: Thursday, July 05, 2001 12:54 PM Subject: Re: PATCH: modify config file loading to allow custom URL types Colin, I modified OptionConverter.java as follows: static public void selectAndConfigure(URL ur

Re: PATCH: modify config file loading to allow custom URL types

2001-07-06 Thread Colin Sampaleanu
EMAIL PROTECTED]> To: "LOG4J Developers Mailing List" <[EMAIL PROTECTED]> Sent: Thursday, July 05, 2001 12:54 PM Subject: Re: PATCH: modify config file loading to allow custom URL types Colin, I modified OptionConverter.java as follows: static public void selectAnd

Re: PATCH: modify config file loading to allow custom URL types

2001-07-05 Thread Ceki Gülcü
Colin, I modified OptionConverter.java as follows: static public void selectAndConfigure(URL url, Hierarchy hierarchy) { String clazz = url.getRef(); if (clazz.indexOf('.') == -1 || clazz.indexOf('/') != -1 || clazz.indexOf('\\') != -1) { LogLog.warn("Suspicious reference