On Oct 5, 2005, at 10:18 AM, Jamis Buck wrote:

> On Oct 5, 2005, at 9:14 AM, Berger, Daniel wrote:
>
>
>> The problem was a hard coded "Format" constant in the Logger class  
>> that
>> defined the format.  This was bad, but not fatal.  The
>> ActiveRecord/Nitro/Needle/Glue, etc, folks made it worse when,  
>> instead
>> of subclassing, they redefined the Logger class and did things like
>> 'remove_const "Format"', among other questionable decisions.  I'm not
>> sure who wrote that code first but it was, unfortunately, copied and
>> used by several authors.
>>
>
> *Ahem*. For what it's worth, Needle *does* subclass Logger. ;) And  
> subclassing is no protection against a change in an internal API.

The *real* problem is that Logger doesn't let us get at that format  
to change it as needed, right?  Can we just patch it to allow that or  
did the change fix this problem?

James Edward Gray II