[squeak-dev] Trunk update works again (Re: Kernel-nice.1127 freezes image (was Another missing snapshot: SmallLand-ColorTheme-fbs.6))

Eliot Miranda eliot.miranda at gmail.com
Mon Feb 5 16:17:40 UTC 2018


Hi All,

> On Feb 5, 2018, at 7:25 AM, Marcel Taeumel <marcel.taeumel at hpi.de> wrote:
> 
> Object >> #currentHand was moved from "Kernel" to "Morphic" back then.

I don't want to flog a dead horse, but...

Do we document the correct approach anywhere easily visible?  When moving code between packages when that code is needed, the correct way is to...

1. Commit the package to which the code is moving, /not/ committing the package from which it has moved yet
2. Commit an update configuration 
3. Commit the package from which the code has moved

Hence on update the package to which the code is moving is loaded first, leaving the other package dirty but the code still present, and then the package from which the code has moved is loaded, leaving the code in place at all times.

This is simple and perhaps non-obvious.  We need to have this, plus similar verbiage about delegate system changes, documented and visible to our users.

> 
> Best,
> Marcel
>> Am 05.02.2018 16:20:04 schrieb Levente Uzonyi <leves at caesar.elte.hu>:
>> 
>> Hi All, 
>> 
>> I went ahead and removed update-nice.422 from the Trunk. The only 
>> difference between update-nice.422 and update-bf.422 was that the former 
>> included Morphic-mt.1350 while the latter referenced Morphic-dtl.1374. 
>> This was a significant difference, because after loading Morphic-mt.1350, 
>> Object >> #currentHand and friends were temporarily removed from the 
>> image, which somehow resulted in the removal of the UI process without an 
>> emergency evaluator ever appearing. I think that's something to be 
>> investigated. 
>> 
>> After this change, I could successfully update the latest Trunk image 
>> available on files.squeak.org. 
>> 
>> I could do this removal only because the members of the Squeak Oversight 
>> Board group on source.squeak.org have not been updated since 2016. For the 
>> very same reason, Nicolas couldn't have done this change. 
>> 
>> Levente 
>> 
>> On Sun, 4 Feb 2018, Bert Freudenberg wrote: 
>> 
>> > Log in to source.squeak.org, trunk repo, "edit configs", pick mine, delete. 
>> > 
>> > Before that you might download the config you're about to delete since there is no backup. 
>> > 
>> > - Bert - 
>> > 
>> > On 2 February 2018 at 21:39, Nicolas Cellier wrote: 
>> > Just in case, I've aligned update-nice.422 with update-bf.422, hope it helps... 
>> > 
>> > 2018-02-02 21:34 GMT+01:00 Nicolas Cellier : 
>> > So the necessary Morphic-dtl.1362.mcz was not in update-nice.422, apologies. 
>> > update-nice.422 should be updated. 
>> > 
>> > Now I see a concurrent update-bf.422, how will it work? 
>> > 
>> > 2018-02-02 21:11 GMT+01:00 Nicolas Cellier : 
>> > Hi, 
>> > I think that it's related to The Trunk: Kernel-dtl.1123.mcz 
>> > http://source.squeak.org/trunk/Kernel-dtl.1123.diff 
>> > 
>> > The missing methods are moved in The Trunk: Morphic-dtl.1362.mcz 
>> > http://source.squeak.org/trunk/Morphic-dtl.1362.diff 
>> > 
>> > So a proper mcm configuration should solve it 
>> > 
>> > 2018-01-30 23:47 GMT+01:00 Fabio Niephaus : 
>> > Clicking the update button another time seemed to have fixed the missing snapshot problem. I tried it a couple of times, but maybe got unlucky with the SqueakSource server. 
>> > Anyway, now installing Kernel-nice.1127 freezes the image and I'm unable to interrupt it. SqueakDebug.log reveals the problem: 
>> > 
>> > --- The full stack --- 
>> > TextMorph(Object)>>doesNotUnderstand: #currentHand 
>> > [] in TextMorph>>createParagraph 
>> > BlockClosure>>ensure: 
>> > TextMorph>>createParagraph 
>> > TextMorph>>paragraph 
>> > TextMorph>>newContents: 
>> > TextMorph>>contentsAsIs: 
>> > MorphicProject>>composeDisplayTextIntoForm: 
>> > DisplayText>>composeForm 
>> > DisplayText>>form 
>> > DisplayText>>displayOn:at:clippingBox:rule:fillColor: 
>> > DisplayText(DisplayObject)>>displayOn:at: 
>> > ByteString(String)>>displayOn:at:textColor:
>> > ByteString(String)>>displayOn:at: 
>> > ByteString(String)>>displayAt: 
>> > Project class>>tryEmergencyEvaluatorForRecovery: 
>> > Project class>>handlePrimitiveError: 
>> > MorphicProject(Object)>>primitiveError: 
>> > MorphicProject(Project)>>handleFatalDrawingError: 
>> >  - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
>> > 
>> > Could someone please look into this? Thanks! 
>> > 
>> > Fabio 
>> > 
>> > image.png 
>> > 
>> > ---------- Forwarded message --------- 
>> > From: Bert Freudenberg 
>> > Date: Tue, Jan 30, 2018 at 11:30 PM 
>> > Subject: Re: [squeak-dev] Another missing snapshot: SmallLand-ColorTheme-fbs.6 
>> > To: The general-purpose Squeak developers list 
>> > 
>> > 
>> > Looks fine to me: 
>> > 
>> > http://source.squeak.org/trunk/SmallLand-ColorTheme-fbs.6.mcz 
>> > 
>> > - Bert - 
>> > 
>> > -- 
>> > 
>> > On 30 January 2018 at 22:03, Fabio Niephaus wrote: 
>> > Hi all, 
>> > 
>> > Updating the trunk base image [1] throws another missing snapshot error. This time it's SmallLand-ColorTheme-fbs.6. Could someone please fix this as it's blocking trunk builds? 
>> > 
>> > Thanks, 
>> > Fabio 
>> > 
>> > [1] http://files.squeak.org/base/Squeak-trunk/base.zip 
>> > 
>> > image.png 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> > 
>> >
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20180205/f1891656/attachment.html>


More information about the Squeak-dev mailing list