search

pharo-project / pharo

Pharo is a dynamic reflective pure object-oriented language supporting live programming inspired by Smalltalk.
http://pharo.org
Other
1.19k stars 352 forks source link

Move package unload hooks from Monticello to RPackage #14073

Open jecisc opened 1 year ago

jecisc commented 1 year ago

There exists some package unload hooks in Pharo using the Manifest with #postUnloadAction and #preUnload.

First it would be nice to unify the API renaming the first one into #postUnload. But the main problem is that this feature is declared in the Kernel but the code executing those actions is in Monticello.

IMO this code should be move to RPackage unloading.

See: MCWorkingCopy>>#unload

Ducasse commented 1 year ago

Agree

MarcusDenker commented 1 year ago

There is https://github.com/pharo-project/pharo/issues/5529 with some ideas (I will close that, we keep this newer one open)