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
