Mariano Martinez Peck wrote
> 
> MCCachedRepository >> marianoDefaultWithDirectory: aDirectory
>     self checkCacheDirectory.
>     ^ default ifNil: [default := self new directory: aDirectory]
> 
I don't think you need to do all this (the checkCacheDirectory and ifNil:
checks are on the old value you're throwing away). How about:

    MCCacheRepository>>default: aFileDirectory

        default := self new directory: aFileDirectory.

Then you won't need to do the reset, and it's general enough to include in
the image w/o extension methods. A setting would be nice too :) I'll rename
my other setting to avoid confusion (I already created an issue).

Issue 5067:     Make package cache location changeable
http://code.google.com/p/pharo/issues/detail?id=5067

I added #default: but not the setting

--
View this message in context: 
http://forum.world.st/Having-a-shared-package-cache-Re-MCDirectoryRepository-defaultDirectoryName-ignored-tp3924814p4167328.html
Sent from the Pharo Smalltalk mailing list archive at Nabble.com.

Reply via email to