DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=28044>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=28044

Patch, Avalon Logging to Commons Logging Conversion.





------- Additional Comments From [EMAIL PROTECTED]  2004-03-30 23:36 -------
Chris,

Good news--I fixed the issue you brought up below--never mind my previous
response.  It turns out that ContainerUtil.enableLogging() [1] is just a
convenience function for setting a logger for the object in question.  Once I
realized what it did, it was trivial to directly to do the same directly via a
Commons-Logger.  I fixed it in two of three places.  In the third
(AbstractPSTranscoder), no logger was defined for this class, nor is any logging
yet done within it.  So enableLogging() below wouldn't do anything for it, and
so I skipped giving it a Commons-Logger at this time.  

I'll upload a new patch.

Thanks,
Glen

[1]
http://avalon.apache.org/framework/api/org/apache/avalon/framework/container/ContainerUtil.html

Jeremias, if you wish I'll happily take care of the issue you mentioned
previously.  It's just you know more about the static loggers (and
centralization of them) than me, so I suspect you can get this looking really
clean much more quickly than I after I apply this patch.  Your choice,
however--I realize you weren't the biggest proponent of this switch to begin
with! ;)

Thanks,
Glen

Reply via email to