On 1 November 2013 13:02, Levente Uzonyi <leves at elte.hu> wrote:
> On Fri, 1 Nov 2013, Nicolas Cellier wrote:
>
>> Please, also remove the preamble and postscript in Graphics, it was a one
>> shot workaround.
>
>
> Those won't be evaluated, unless they have changed.

But it's a good practice to follow, because otherwise someone else
hacking on Graphics that needs a preamble doesn't have to wonder about
whether or not the existing code's still needed.

frank

> Levente
>
>
>>
>>
>> 2013/11/1 Frank Shearar <frank.shearar at gmail.com>
>>       The trick is to write the mail and then go to bed. In the morning,
>> someone (some people) will have solved your problems! I'll publish
>>       two separate mcms.
>>
>>       Thanks!
>>
>>       frank
>>
>>       On 01 Nov 2013, at 0:44, Levente Uzonyi <leves at elte.hu> wrote:
>>
>>       > I should read all mails before I start replying. :)
>>       >
>>       >
>>       > Levente
>>       >
>>       > On Thu, 31 Oct 2013, Nicolas Cellier wrote:
>>       >
>>       >> Yes, but the other way is to commit a new mcm with the new
>> UIManagers, but without Graphics changes.
>>       >> You then do not need to mess with load order... Unless I missed
>> something.
>>       >> 2013/10/31 Frank Shearar <frank.shearar at gmail.com>
>>       >>      On 31 October 2013 22:19,  <commits at source.squeak.org>
>> wrote:
>>       >>      > Frank Shearar uploaded a new version of ToolBuilder-Kernel
>> to project The Trunk:
>>       >>      >
>> http://source.squeak.org/trunk/ToolBuilder-Kernel-fbs.58.mcz
>>       >>      >
>>       >>      > ==================== Summary ====================
>>       >>      >
>>       >>      > Name: ToolBuilder-Kernel-fbs.58
>>       >>      > Author: fbs
>>       >>      > Time: 31 October 2013, 10:18:57.495 pm
>>       >>      > UUID: f4158a78-a770-0548-a26b-a9cfd0ea9beb
>>       >>      > Ancestors: ToolBuilder-Kernel-dtl.57
>>       >>      >
>>       >>      > Step 1 of moving control over display depth to the
>> UIManagers.
>>       >>      >
>>       >>      > =============== Diff against ToolBuilder-Kernel-dtl.57
>> ===============
>>       >> So I think the next step is to make the DisplayScreen methods
>> forward
>>       >> entirely to UIManagers. But getting this wrong means bad things.
>> So I
>>       >> thought I'd check first that this will work out:
>>       >> * make the change
>>       >> * commit Graphics
>>       >> * change the update configuration map to load the UIManagers
>> before Graphics.
>>       >> * copy Graphics to trunk
>>       >> Think it'll work?
>>       >> frank