On 31 December 2013 23:49, Chris Muller <asqueaker at gmail.com> wrote:
> On Tue, Dec 31, 2013 at 5:13 PM, Frank Shearar <frank.shearar at gmail.com> wrote:
>> On 31 December 2013 22:27,  <commits at source.squeak.org> wrote:
>>> A new version of ReleaseBuilder was added to project The Inbox:
>>> http://source.squeak.org/inbox/ReleaseBuilder-cmm.105.mcz
>>>
>>> ==================== Summary ====================
>>>
>>> Name: ReleaseBuilder-cmm.105
>>> Author: cmm
>>> Time: 31 December 2013, 4:27:44.493 pm
>>> UUID: 0bae46e0-1d72-49c3-830c-5dc2fa2b4958
>>> Ancestors: ReleaseBuilder-cmm.104
>>>
>>> Please let all internal image work and modifications for building our releases be delegated to ReleaseBuilder.
>>>
>>> =============== Diff against ReleaseBuilder-cmm.104 ===============
>>>
>>> Item was changed:
>>>   ----- Method: ReleaseBuilder class>>prepareNewBuild: (in category 'scripts') -----
>>>   prepareNewBuild: anMCRepository
>>>         "ReleaseBuilderTrunk prepareNewBuild"
>>>         "Prepare everything that should be done for a new image build"
>>> +       MCMcmUpdater updateMissingPackages: true.
>>> +       MCMcmUpdater enableUpdatesForAllPackages.
>>> +       Workspace allInstances do: [ : each | each topView delete].
>>
>> Duly removed from squeak-ci's release.st.
>>
>>> +       MorphicProject initialize.
>>
>> Why? I guess to set the background from MorphicProject class >>
>> #defaultBackgroundForm.
>
> Yes.
>
>> Why store an image in a string inside the
>> image?
>
> Well, this is a separate discussion which Nicolas also recently asked.
>  Background and icon data represented in big String literals is
> nothing new and everyone recognizes its an undesirable redundancy;
> something we should talk about eliminating in 4.6.

Fair enough. I was also thinking of Nicolas' experiences.

> Let's just stay with the old way for 4.5 so we can get it done and
> everyone can see where the release background comes from.  Notice I've
> at least refreshed the form with one that is a lot smaller than the
> old one.

Well, it's your release to manage, so it's up to you as to how exactly
you want to run things. I suppose I'm just a bit miffed that you're
not using the process I spent time developing.

>> It looks like you're jumping through hoops just to avoid
>> touching anything outside the image.
>
> No.

OK.

frank

>> (release.st pulls in the image from the squeak-ci repository. The
>> image is currently this guy -
>> https://github.com/squeak-smalltalk/squeak-ci/blob/de815b8722284e8095c0ed35ee29606f9c19dbd8/resources/wallpaper.png)
>>
>> frank
>>
>