> -----Original Message-----
> From: Stephen McConnell [mailto:[EMAIL PROTECTED] 
> Sent: Tuesday, February 24, 2004 3:21 PM
> To: Avalon Developers List
> Subject: Re: Duplicate LogKitLogger?
> 
> 
> Carsten:
> 
> Generally speaking I'm in favour of removing the class.  Just 
> for reference - is this the only reference to framework?  

Yes, it is.

> If 
> we do remove the class we would need to bump the logkit 
> version to 2.0 as this is a change that would break backward 
> compatability.
> 
Hmm, yes in theory you're right, but I really think that noone
ever used these classes, so 1.3 with a visible remark would imho
do it as well.

Carsten

> Stephen.
> 
> 
> Carsten Ziegeler wrote:
> 
> > Both, the LogKit and the avalon-framework/impl contain the 
> > LogKit2AvalonLoggerAdapter and the LogKit(Avalon)Logger.
> > 
> > Now these classes are nearly identical, so I guess one set 
> of them is 
> > superfluous.
> > 
> > If possible, I would suggest to remove the classes from LogKit as 
> > these two classes have a reference from the LogKit to the 
> > avalon-framework. Of course, the framework has references to the 
> > LogKit, so we end up with circular dependencies :(
> > 
> > What do you think?
> > 
> > Carsten
> > 
> > 
> > 
> > 
> ---------------------------------------------------------------------
> > To unsubscribe, e-mail: [EMAIL PROTECTED] For 
> > additional commands, e-mail: [EMAIL PROTECTED]
> > 
> > 
> 
> 
> -- 
> 
> |------------------------------------------------|
> | Magic by Merlin                                |
> | Production by Avalon                           |
> |                                                |
> | http://avalon.apache.org/merlin                |
> | http://dpml.net/merlin/distributions/latest    |
> |------------------------------------------------|
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED] For 
> additional commands, e-mail: [EMAIL PROTECTED]
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to