Absolutely. I'm certainly not trying to say "stop!". Just pointing out
that package dependencies are important, and need to be tightly
controlled (especially given the crazy hairball dependency graph that
we currently have).

frank

On 27 September 2013 12:00, Nicolas Cellier
<nicolas.cellier.aka.nice at gmail.com> wrote:
> In fact I think that TextComposer was formerly in Morphic, so that would
> explain why you once needed to break the dependecy.
> But you then resolved the problem by moving TextComposer to Graphics.
> So this left over the methods using TextComposer in Morphic...
> It doesn't matter that much, we can have several trials and errors to
> improve the system...
> ...it's just that changing core code that might block the UI is more
> challenging when it is spanning several package (w.r.t. load order etc...)
>
>
> 2013/9/27 Frank Shearar <frank.shearar at gmail.com>
>>
>> On 27 September 2013 01:06,  <commits at source.squeak.org> wrote:
>> > Nicolas Cellier uploaded a new version of Morphic to project The Trunk:
>> > http://source.squeak.org/trunk/Morphic-nice.691.mcz
>> >
>> > ==================== Summary ====================
>> >
>> > Name: Morphic-nice.691
>> > Author: nice
>> > Time: 27 September 2013, 2:05:45.131 am
>> > UUID: e25c6256-260f-4f90-9826-bc204f8c6a30
>> > Ancestors: Morphic-nice.690
>> >
>> > Give some methods back the the (Multi)CharacterScanner
>> >
>> > =============== Diff against Morphic-nice.690 ===============
>>
>> I'm the first to admit that my recategorising of methods is sometimes
>> rather arbitrary, in my work on decoupling packages. It's work running
>> the PackageDependencyTest when making these changes, so that we don't
>> start re-adding cycles.
>>
>> For instance, one of the recent changes (not sure which (*)) has added
>> a System -> Installer-Core dependency through referencing Installer.
>>
>> (*) Diff
>> http://build.squeak.org/job/SqueakTrunk/539/artifact/target/TrunkImage.manifest
>> and
>> http://build.squeak.org/job/SqueakTrunk/540/artifact/target/TrunkImage.manifest
>> and you'll see the package suspects.
>>
>> frank
>>
>
>
>
>