[squeak-dev] Re: The Trunk: Collections-dtl.276.mcz

Levente Uzonyi leves at elte.hu
Sun Jan 10 18:43:20 UTC 2010


On Sun, 10 Jan 2010, Andreas Raab wrote:

> David T. Lewis wrote:
>> On Sun, Jan 10, 2010 at 03:09:10AM +0100, Levente Uzonyi wrote:
>>> Why aren't these methods in the 311Deprecated package?
>> 
>> Good question, and I'm not sure which is better in this case. I think
>> the choice should be driven by whether we view these as deprecated,
>> implying that they would be removed perhaps a year or so from now, or
>> if they are intended as compatability methods that might stay in the
>> image for a longer period of time.
>
> I agree. The reason why I put findElementOrNil: back right away was that it's 
> being used by DynamicBindings of which a multitude of versions exists in many 
> places on the web. It's virtually impossible to fix all the existing 
> versions.
>
>> I'm pretty sure that the folks doing e.g. Seaside are going to feel
>> the same way, so we need to be careful to provide compatibility
>> methods where appropriate, but not go overboard with it and clutter
>> up the image with junk. So, to return to the original question,
>> I'm not sure if these methods should be moved to 311Deprecated,
>> but if the intent is to keep them for say five years for compatibility,
>> then no they should not be moved.
>
> +1 on all of it. We should get rid of these methods whenever we can but we 
> also need to take into account the existence of external packages. In the 
> case of findElementOrNil: I first found (and fixed) a usage in Tweak, then I 
> found (and fixed) a usage in DynamicBindings on http://source.squeak.org/ss 
> but then I ran into one usage via DynamicBindings in a package loaded three 
> indirections deep via Metacello. That's what pushed it over the edge for me - 
> clearly there's more than one version out there and it's hard to predict 
> which one people will attempt to load.

There's a DynamicBindings repositiory at squeaksource.com 
(http://squeaksource.com/DynamicBindings.html ) which seems to be 
abandoned. AFAIK 
the package is maintained at http://squeaksource.com/KomHttpServer.html .


Levente

>
> Cheers,
>  - Andreas
>
>



More information about the Squeak-dev mailing list