On Wed, Dec 7, 2011 at 2:48 AM, Sean P. DeNigris <[email protected]>wrote:

>
> 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
>
>
Yes, and you can also directly execute:

MCCacheRepository default
directory: (FileDirectory on:  '/Users/mariano/Pharo/sharedRepo').




> --
> 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.
>
>


-- 
Mariano
http://marianopeck.wordpress.com

Reply via email to