[Pharo-project] Issue 3508 in pharo: cleanUpForProduction should really unload 'Graphics-Resources'
Status: Accepted Owner: marianopeck Labels: Milestone-1.2 New issue 3508 by marianopeck: cleanUpForProduction should really unload 'Graphics-Resources' http://code.google.com/p/pharo/issues/detail?id=3508 now cleanUpForProduction is doing: (MCPackage named: 'GraphicsResources') unload. but the package doesn't exist. So, it should be: (MCPackage named: 'Graphics-Resources') unload. Pharo core version: 12292
Updates: Status: Fixed Comment #1 on issue 3508 by marianopeck: cleanUpForProduction should really unload 'Graphics-Resources' http://code.google.com/p/pharo/issues/detail?id=3508 (No comment was entered for this change.)
Updates: Status: Comment Comment #2 on issue 3508 by adrian.lienhard: cleanUpForProduction should really unload 'Graphics-Resources' http://code.google.com/p/pharo/issues/detail?id=3508 No! :) This is on purpose. There is no class category GraphicsResources but there are class extensions *graphicsresources (you can see this when you browse the package with MC). The reason I did this is to be able to unload these methods without making the package 'Graphics' dirty. These graphic resources are not needed unless you want to recreate the icons. Adrian
Comment #3 on issue 3508 by marianopeck: cleanUpForProduction should really unload 'Graphics-Resources' http://code.google.com/p/pharo/issues/detail?id=3508 Hi Adrian. Are you checking in Pharo1.2 ? Because I don't see any package GraphicsResources and in addition, if I browse Graphics-Resources I see the class extensions with the correct caegory *GraphicsResources Finally, I don't see a package Graphics, but instead all Graphics-Something cheers
Updates: Status: Fixed Comment #4 on issue 3508 by adrian.lienhard: cleanUpForProduction should really unload 'Graphics-Resources' http://code.google.com/p/pharo/issues/detail?id=3508 Hi Mariano. Here too, I checked with a 1.1.1 image. In 1.2 also the graphics package structure was changed (in a good way). Apparently this was done with issue ##2105. So the change you propose with this issue makes sense! Sorry for the confusion... Adrian
Comment #5 on issue 3508 by marianopeck: cleanUpForProduction should really unload 'Graphics-Resources' http://code.google.com/p/pharo/issues/detail?id=3508 Thanks adrian for the warning :) In inbox: Name: SLICE-Issue-3508-cleanUpForProductionShouldRemoveGraphicsResources-MarianoMartinezPeck.1 Author: MarianoMartinezPeck Time: 6 January 2011, 8:03:33 pm UUID: dc80b20c-ddce-41d2-b9e6-48e4c4fe5dc5 Ancestors: Dependencies: ScriptLoader12-MarianoMartinezPeck.365 fix to issue 3508
Updates: Status: Closed Comment #6 on issue 3508 by marcus.denker: cleanUpForProduction should really unload 'Graphics-Resources' http://code.google.com/p/pharo/issues/detail?id=3508 12293
participants (1)
-
pharo＠googlecode.com