On Wed, Dec 7, 2011 at 2:48 AM, Sean P. DeNigris <sean@clipperadams.com> 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