On 30 December 2013 21:18, Chris Muller <asqueaker at gmail.com> wrote:
> I thought we said Collections was one of the triad of "Core" classes,
> along with Kernel and Exceptions.

Yep, at least until it's ripped apart. (Remember, we'd discussed
moving those parts of Collection that Kernel absolutely requires
(Kernel itself needs to go on a diet too, after all) into Kernel, and
keep Collections containing the other generally-useful-but-not-Kernel
stuff.)

> That means Tools, with all of it's 30+ "Browser" classes and more must
> be part of that core system too?

:) No, of course not.

> Collections _depending_ on Tools.  Something doesn't seem right here..

Hence "(Eventually this dependency should disappear.)" But since this
has stirred up conversation, let's figure out where this method
_should_ go. It's hooked into compilation stuff, so maybe it should go
into Compiler, or a Compiler-related package rather, along with
ClassDescription >> #compile:classified:withStamp:notifying:logSource:
and all the myriad other bits that use a Compiler.

But hey, if there's a better place than SystemBrowser for a preference
called "browseWithPrettyPrint", I'm happy to move it.

frank

> On Sun, Dec 29, 2013 at 4:20 PM,  <commits at source.squeak.org> wrote:
>> Frank Shearar uploaded a new version of Tests to project The Trunk:
>> http://source.squeak.org/trunk/Tests-fbs.280.mcz
>>
>> ==================== Summary ====================
>>
>> Name: Tests-fbs.280
>> Author: fbs
>> Time: 29 December 2013, 10:20:48.376 pm
>> UUID: 6db4237c-c2f3-3440-9357-b77b8cb6f576
>> Ancestors: Tests-fbs.279
>>
>> Collections now depends on Tools, because of the recent move of #browseWithPrettyPrint preference from Preferences to SystemBrowser. (Eventually this dependency should disappear.)
>>
>> =============== Diff against Tests-fbs.279 ===============
>>
>> Item was changed:
>>   ----- Method: PackageDependencyTest>>testCollections (in category 'tests') -----
>>   testCollections
>>         self testPackage: 'Collections' dependsExactlyOn: #(
>>                 Compiler
>>                 Kernel
>>                 Files
>>                 Graphics
>>                 Multilingual
>>                 System
>>                 'ToolBuilder-Kernel'
>> +               Tools
>>         ).!
>>
>>
>