Deprecated long overdue in 5.3

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

Deprecated long overdue in 5.3

Karl Ramberg
Hi, 
We have several methods that have been deprecated for several releases. We should prepare for removal of these now in the beginning of 5.3 


Best,
Karl 


Reply | Threaded
Open this post in threaded view
|

Re: Deprecated long overdue in 5.3

Eliot Miranda-2
Hi Karl,

> On Oct 18, 2018, at 3:29 AM, karl ramberg <[hidden email]> wrote:
>
> Hi,
> We have several methods that have been deprecated for several releases. We should prepare for removal of these now in the beginning of 5.3

+1.  We should remove the 4.x packages ASAP.  But what’s the right way to remove a package?  I guess if one wants to remove an MNDeprecated package one can add a script to the MN+1Deprecated package.  Is there a better way?

> Best,
> Karl

Reply | Threaded
Open this post in threaded view
|

Re: Deprecated long overdue in 5.3

Levente Uzonyi
On Thu, 18 Oct 2018, Eliot Miranda wrote:

> Hi Karl,
>
>> On Oct 18, 2018, at 3:29 AM, karl ramberg <[hidden email]> wrote:
>>
>> Hi,
>> We have several methods that have been deprecated for several releases. We should prepare for removal of these now in the beginning of 5.3
>
> +1.  We should remove the 4.x packages ASAP.  But what’s the right way to remove a package?  I guess if one wants to remove an MNDeprecated package one can add a script to the MN+1Deprecated package.  Is there a better way?

Since you have to bump the version number of the Squeak-Version
package anyway when you unload a package to ensure the monotonicity of the
global version number, the common practice is to add the unload script as
the postscript of that package. A line such the one below should do it:

(MCPackage named: '<name of the package to unload') unload

Levente

P.S.: do not remove 39deprecated yet. It's minimal, and it contains two
methods that we need if we want to migrate from SharedQueue to
SharedQueue2.

>
>> Best,
>> Karl

cbc
Reply | Threaded
Open this post in threaded view
|

Re: Deprecated long overdue in 5.3

cbc


On Thu, Oct 18, 2018 at 6:47 AM Levente Uzonyi <[hidden email]> wrote:

P.S.: do not remove 39deprecated yet. It's minimal, and it contains two
methods that we need if we want to migrate from SharedQueue to
SharedQueue2.

Let's make this choice now instead of keeping 3.9 around for a few more years?  Either we make the switch, or we finally abandon it.

Alternatively, if we HAVE to keep delaying this choice, could we at least move those 2 methods up to a 5.x deprecation package, and remove 39deprecated?  any reason this is a bad idea?

-cbc


Reply | Threaded
Open this post in threaded view
|

Re: Deprecated long overdue in 5.3

timrowledge
Yes, please let's get some of the deprecation actually done - or what's the point?

And as a side question what can we do about cases where we have not-very-nice class names because of making replacement versions? 'SharedQueue2' isn't the best of names. I'd much prefer the deprecation process replaced the old with new, though I understand there are potential issues.

tim
--
tim Rowledge; [hidden email]; http://www.rowledge.org/tim
Strange Opcodes: ZZZZZZZZZZZZ: enter sleep mode