On Mon, Jul 08, 2013 at 08:01:32AM +0100, Frank Shearar wrote:
> On 8 July 2013 06:15,  <commits at source.squeak.org> wrote:
> > David T. Lewis uploaded a new version of System to project The Trunk:
> > http://source.squeak.org/trunk/System-dtl.560.mcz
> >
> > ==================== Summary ====================
> >
> > Name: System-dtl.560
> > Author: dtl
> > Time: 8 July 2013, 1:15:09.698 am
> > UUID: f9176e6f-41f5-4bec-a2d0-a15717fabdeb
> > Ancestors: System-fbs.559
> >
> > Add ST80Tools and ST80Tests to the list of packages to be unloaded in #unloadAllKnownPackages. These are new package categories that should be unloaded as part of MVC unloading.
> >
> > =============== Diff against System-fbs.559 ===============
> >
> > Item was changed:
> >   ----- Method: SmalltalkImage>>unloadAllKnownPackages (in category 'shrinking') -----
> >   unloadAllKnownPackages
> >         "Unload all packages we know how to unload and reload"
> >
> >         "Prepare unloading"
> >         Smalltalk zapMVCprojects.
> >         Flaps disableGlobalFlaps: false.
> >         StandardScriptingSystem removeUnreferencedPlayers.
> >         Project removeAllButCurrent.
> >         #('Morphic-UserObjects' 'EToy-UserObjects' 'Morphic-Imported' )
> >                 do: [:each | SystemOrganization removeSystemCategory: each].
> >         Smalltalk at: #ServiceRegistry ifPresent:[:aClass|
> >                 SystemChangeNotifier uniqueInstance
> >                         noMoreNotificationsFor: aClass.
> >         ].
> >         World removeAllMorphs.
> >
> >         "Go unloading"
> >         #(      'ReleaseBuilder' 'ScriptLoader'
> >                 '311Deprecated' '39Deprecated'
> >                 'Universes' 'SMLoader' 'SMBase' 'Installer-Core'
> >                 'VersionNumberTests' 'VersionNumber'
> >                 'Services-Base' 'PreferenceBrowser' 'Nebraska'
> > +               'ToolBuilder-MVC' 'ST80Tools' 'ST80Tests' 'ST80'
> 
> Thanks, Dave. I remembered to edit ReleaseBuilder class >>
> #loadWellKnownPackages, but not this guy!
> 
> Oh, and 45Deprecated can go here too.
> 
> But... how can we verify that this guy still works?
>

Good question. At this point it is clearly a manual verification. Try doing
the unload, see if the resulting image still works and does not have
references to the unloaded packages. It would be nice if we could automate
that in the form of a test, but I don't know how to do it.

Dave