> On 13 Sep 2016, at 13:15, Vitor Medina Cruz <vitormc...@gmail.com> wrote: > > Humm, why not? When I load the configuration it loads several other packages, > wouldn't that need a cleanup too? When I first see that I could cleanup an > image my first thought was to put it in a production state with my code, then > I would cleanup to leave it as lean as possible. Isn't that correct? > > If I should not clean after load my packages why provide the functionality? > Why not provide clean images to be used in production so that people could > download and load it's ready to production code? >
No, the #cleanUpForProduction just unloads tests, examples and things like that… it only touches loaded code in the sense that it unloads all packages named -Tests and -Examples. But if the external package happens to be structured in a way that this is bad, it would brake. I never expected this to be run *after* more packages are loaded, just *before*. What it does right now it this (you can see that it could be simpler if our packages are better structured): cleanUpForProduction "self new cleanUpForProduction" "delete logo" PolymorphSystemSettings showDesktopLogo: false. self packagesForCleanUpInProduction do: [ :each | (MCPackage named: each) unload ]. self helpPackages do: [ :each | (MCPackage named: each) unload ]. self testPackages do: [ :each | (MCPackage named: each) unload ]. self examplePackages do: [ :each | (MCPackage named: each) unload ]. (MCPackage named: 'ScriptLoader40') unload. (MCPackage named: 'MonticelloMocks') unload. (MCPackage named: 'ToolsTest') unload. (MCPackage named: 'Announcements-Tests-Core') unload. (MCPackage named: 'AST-Tests-Core') unload. (MCPackage named: 'AST-Interpreter-Test') unload. (MCPackage named: 'Ring-Tests-Containers') unload. (MCPackage named: 'Ring-Tests-Kernel') unload. (MCPackage named: 'Ring-Tests-Monticello') unload. (MCPackage named: 'Regex-Tests-Core') unload. (MCPackage named: 'Refactoring-Tests-Changes') unload. (MCPackage named: 'Refactoring-Tests-Core') unload. (MCPackage named: 'Refactoring-Tests-Critics') unload. (MCPackage named: 'Refactoring-Tests-Environment') unload. (MCPackage named: 'FileSystem-Tests-Core') unload. (MCPackage named: 'FileSystem-Tests-Disk') unload. (MCPackage named: 'FileSystem-Tests-Memory') unload. (MCPackage named: 'GT-Tests-Inspector') unload. (MCPackage named: 'Glamour-Tests-Core') unload. (MCPackage named: 'Glamour-Tests-Morphic') unload. (MCPackage named: 'Glamour-Tests-Resources') unload. (MCPackage named: 'Glamour-Tests-Rubric') unload. (MCPackage named: 'Versionner-Core-Model') unload. (MCPackage named: 'Versionner-Core-DependenciesModel') unload. (MCPackage named: 'Versionner-Core-Announcements') unload. (MCPackage named: 'Versionner-Core-Commands') unload. (MCPackage named: 'Versionner-Spec-Browser') unload. (MCPackage named: 'Versionner-Tests-Core-Commands') unload. (MCPackage named: 'Versionner-Tests-Core-DependenciesModel') unload. (MCPackage named: 'Versionner-Tests-Core-Model') unload. (MCPackage named: 'Versionner-Tests-Resources') unload. self cleanUpForRelease. Marcus