The Trunk: SMLoader-fbs.78.mcz

> Frank Shearar uploaded a new version of SMLoader to project The Trunk:
> http://source.squeak.org/trunk/SMLoader-fbs.78.mcz
>
> ==================== Summary ====================
>
> Name: SMLoader-fbs.78
> Author: fbs
> Time: 25 February 2013, 9:27:11.959 pm
> UUID: cd8023fe-60a1-4dd9-a7dd-60e8ce7f4ac5
> Ancestors: SMLoader-cmm.77
>
> When installing WebClient, use the most recent version.
>
> =============== Diff against SMLoader-cmm.77 ===============
>
> Item was changed:
>   SystemOrganization addCategory: #SMLoader!
> - SystemOrganization addCategory: #'SMLoader-Models'!
>
> Item was changed:
>   ----- Method: SMClient class>>installWebClient (in category 'private') -----
>   installWebClient
> +       Installer ss
> +               project: 'WebClient';
> +               install: 'WebClient-Core-ar.92.mcz'!
> -       (Installer repository: 'http://www.squeaksource.com/WebClient')
> -               install: 'WebClient-Core-ar.83'!
>
>

> Frank Shearar uploaded a new version of SMLoader to project The Trunk:
> http://source.squeak.org/trunk/SMLoader-fbs.78.mcz
>
> ==================== Summary ====================
>
> Name: SMLoader-fbs.78
> Author: fbs
> Time: 25 February 2013, 9:27:11.959 pm
> UUID: cd8023fe-60a1-4dd9-a7dd-60e8ce7f4ac5
> Ancestors: SMLoader-cmm.77
>
> When installing WebClient, use the most recent version.
>
> =============== Diff against SMLoader-cmm.77 ===============
>
> Item was changed:
>  SystemOrganization addCategory: #SMLoader!
> - SystemOrganization addCategory: #'SMLoader-Models'!
>
> Item was changed:
>  ----- Method: SMClient class>>installWebClient (in category 'private') -----
>  installWebClient
> + Installer ss
> + project: 'WebClient';
> + install: 'WebClient-Core-ar.92.mcz'!

> Am 25.02.2013 um 21:27 schrieb [hidden email]:
>
>> Frank Shearar uploaded a new version of SMLoader to project The Trunk:
>> http://source.squeak.org/trunk/SMLoader-fbs.78.mcz
>>
>> ==================== Summary ====================
>>
>> Name: SMLoader-fbs.78
>> Author: fbs
>> Time: 25 February 2013, 9:27:11.959 pm
>> UUID: cd8023fe-60a1-4dd9-a7dd-60e8ce7f4ac5
>> Ancestors: SMLoader-cmm.77
>>
>> When installing WebClient, use the most recent version.
>>
>> =============== Diff against SMLoader-cmm.77 ===============
>>
>> Item was changed:
>>  SystemOrganization addCategory: #SMLoader!
>> - SystemOrganization addCategory: #'SMLoader-Models'!
>>
>> Item was changed:
>>  ----- Method: SMClient class>>installWebClient (in category 'private') -----
>>  installWebClient
>> +     Installer ss
>> +             project: 'WebClient';
>> +             install: 'WebClient-Core-ar.92.mcz'!
>
> Shouldn't this be WebClient-Core-ar.92 without the .mcz?

> I'm glad you left it a hard-coded version (92) so that it will stay stable.
>
>
> On Mon, Feb 25, 2013 at 3:27 PM,  <[hidden email]> wrote:
>> Frank Shearar uploaded a new version of SMLoader to project The Trunk:
>> http://source.squeak.org/trunk/SMLoader-fbs.78.mcz
>>
>> ==================== Summary ====================
>>
>> Name: SMLoader-fbs.78
>> Author: fbs
>> Time: 25 February 2013, 9:27:11.959 pm
>> UUID: cd8023fe-60a1-4dd9-a7dd-60e8ce7f4ac5
>> Ancestors: SMLoader-cmm.77
>>
>> When installing WebClient, use the most recent version.
>>
>> =============== Diff against SMLoader-cmm.77 ===============
>>
>> Item was changed:
>>   SystemOrganization addCategory: #SMLoader!
>> - SystemOrganization addCategory: #'SMLoader-Models'!
>>
>> Item was changed:
>>   ----- Method: SMClient class>>installWebClient (in category 'private') -----
>>   installWebClient
>> +       Installer ss
>> +               project: 'WebClient';
>> +               install: 'WebClient-Core-ar.92.mcz'!
>> -       (Installer repository: 'http://www.squeaksource.com/WebClient')
>> -               install: 'WebClient-Core-ar.83'!
>>
>>
>

> On 26 February 2013 06:38, Tobias Pape <[hidden email]> wrote:
>> Am 25.02.2013 um 21:27 schrieb [hidden email]:
>>
>>> Frank Shearar uploaded a new version of SMLoader to project The Trunk:
>>> http://source.squeak.org/trunk/SMLoader-fbs.78.mcz
>>>
>>> ==================== Summary ====================
>>>
>>> Name: SMLoader-fbs.78
>>> Author: fbs
>>> Time: 25 February 2013, 9:27:11.959 pm
>>> UUID: cd8023fe-60a1-4dd9-a7dd-60e8ce7f4ac5
>>> Ancestors: SMLoader-cmm.77
>>>
>>> When installing WebClient, use the most recent version.
>>>
>>> =============== Diff against SMLoader-cmm.77 ===============
>>>
>>> Item was changed:
>>> SystemOrganization addCategory: #SMLoader!
>>> - SystemOrganization addCategory: #'SMLoader-Models'!
>>>
>>> Item was changed:
>>> ----- Method: SMClient class>>installWebClient (in category 'private') -----
>>> installWebClient
>>> +     Installer ss
>>> +             project: 'WebClient';
>>> +             install: 'WebClient-Core-ar.92.mcz'!
>>
>> Shouldn't this be WebClient-Core-ar.92 without the .mcz?
>
> Yes, it should, but without the .mcz you'd end up loading
> WebClient-Core-ar.921.mcz by accident, were there such a version.

> Am 26.02.2013 um 08:19 schrieb Frank Shearar <[hidden email]>:
>
>> On 26 February 2013 06:38, Tobias Pape <[hidden email]> wrote:
>>> Am 25.02.2013 um 21:27 schrieb [hidden email]:
>>>
>>>> Frank Shearar uploaded a new version of SMLoader to project The Trunk:
>>>> http://source.squeak.org/trunk/SMLoader-fbs.78.mcz
>>>>
>>>> ==================== Summary ====================
>>>>
>>>> Name: SMLoader-fbs.78
>>>> Author: fbs
>>>> Time: 25 February 2013, 9:27:11.959 pm
>>>> UUID: cd8023fe-60a1-4dd9-a7dd-60e8ce7f4ac5
>>>> Ancestors: SMLoader-cmm.77
>>>>
>>>> When installing WebClient, use the most recent version.
>>>>
>>>> =============== Diff against SMLoader-cmm.77 ===============
>>>>
>>>> Item was changed:
>>>> SystemOrganization addCategory: #SMLoader!
>>>> - SystemOrganization addCategory: #'SMLoader-Models'!
>>>>
>>>> Item was changed:
>>>> ----- Method: SMClient class>>installWebClient (in category 'private') -----
>>>> installWebClient
>>>> +     Installer ss
>>>> +             project: 'WebClient';
>>>> +             install: 'WebClient-Core-ar.92.mcz'!
>>>
>>> Shouldn't this be WebClient-Core-ar.92 without the .mcz?
>>
>> Yes, it should, but without the .mcz you'd end up loading
>> WebClient-Core-ar.921.mcz by accident, were there such a version.
>
>
> Then I would like to declare Installer broken. ;)
> Rephrased: This would be unexpected behavior for me

> On 26 February 2013 07:23, Tobias Pape <[hidden email]> wrote:
>> Am 26.02.2013 um 08:19 schrieb Frank Shearar <[hidden email]>:
>>
>>> On 26 February 2013 06:38, Tobias Pape <[hidden email]> wrote:
>>>
>>> Yes, it should, but without the .mcz you'd end up loading
>>> WebClient-Core-ar.921.mcz by accident, were there such a version.
>>
>>
>> Then I would like to declare Installer broken. ;)
>> Rephrased: This would be unexpected behavior for me
>
> If you raise a bug I'll remember to fix it :) Otherwise my
> disease-riddled brain will likely forget.

> Am 26.02.2013 um 08:28 schrieb Frank Shearar <[hidden email]>:
>
>> On 26 February 2013 07:23, Tobias Pape <[hidden email]> wrote:
>>> Am 26.02.2013 um 08:19 schrieb Frank Shearar <[hidden email]>:
>>>
>>>> On 26 February 2013 06:38, Tobias Pape <[hidden email]> wrote:
>>>>
>>>> Yes, it should, but without the .mcz you'd end up loading
>>>> WebClient-Core-ar.921.mcz by accident, were there such a version.
>>>
>>>
>>> Then I would like to declare Installer broken. ;)
>>> Rephrased: This would be unexpected behavior for me
>>
>> If you raise a bug I'll remember to fix it :) Otherwise my
>> disease-riddled brain will likely forget.
>
>
> you mean on mantis?
> To be frank, I have no idea how to.
> I find mantis most puzzeling :/

> On 26 February 2013 07:43, Tobias Pape <[hidden email]> wrote:
>> Am 26.02.2013 um 08:28 schrieb Frank Shearar <[hidden email]>:
>>
>>> On 26 February 2013 07:23, Tobias Pape <[hidden email]> wrote:
>>>> Am 26.02.2013 um 08:19 schrieb Frank Shearar <[hidden email]>:
>>>>
>>>>> On 26 February 2013 06:38, Tobias Pape <[hidden email]> wrote:
>>>>>
>>>>> Yes, it should, but without the .mcz you'd end up loading
>>>>> WebClient-Core-ar.921.mcz by accident, were there such a version.
>>>>
>>>>
>>>> Then I would like to declare Installer broken. ;)
>>>> Rephrased: This would be unexpected behavior for me
>>>
>>> If you raise a bug I'll remember to fix it :) Otherwise my
>>> disease-riddled brain will likely forget.
>>
>>
>> you mean on mantis?
>> To be frank, I have no idea how to.
>> I find mantis most puzzeling :/
>
> Really? It's very retro chic, but I've always found its interface to
> be more or less navigable. It _has_ been a very long time since I
> learned Mantis though, to be fair.
>
> So I raised the ticket for you: http://bugs.squeak.org/view.php?id=7742

> On 26 February 2013 07:43, Tobias Pape <[hidden email]> wrote:
>> Am 26.02.2013 um 08:28 schrieb Frank Shearar <[hidden email]>:
>>
>>> On 26 February 2013 07:23, Tobias Pape <[hidden email]> wrote:
>>>
>>> If you raise a bug I'll remember to fix it :) Otherwise my
>>> disease-riddled brain will likely forget.
>>
>>
>> you mean on mantis?
>> To be frank, I have no idea how to.
>> I find mantis most puzzeling :/
>
> Really? It's very retro chic, but I've always found its interface to
> be more or less navigable. It _has_ been a very long time since I
> learned Mantis though, to be fair.
>
> So I raised the ticket for you: http://bugs.squeak.org/view.php?id=7742

> On 26 February 2013 06:38, Tobias Pape <[hidden email]> wrote:
>> Am 25.02.2013 um 21:27 schrieb [hidden email]:
>>
>>> installWebClient
>>> +     Installer ss
>>> +             project: 'WebClient';
>>> +             install: 'WebClient-Core-ar.92.mcz'!
>>
>> Shouldn't this be WebClient-Core-ar.92 without the .mcz?
>
> Yes, it should, but without the .mcz you'd end up loading
> WebClient-Core-ar.921.mcz by accident, were there such a version.
>
> frank

>
> On 2013-02-26, at 08:19, Frank Shearar <[hidden email]> wrote:
>
>> On 26 February 2013 06:38, Tobias Pape <[hidden email]> wrote:
>>> Am 25.02.2013 um 21:27 schrieb [hidden email]:
>>>
>>>> installWebClient
>>>> +     Installer ss
>>>> +             project: 'WebClient';
>>>> +             install: 'WebClient-Core-ar.92.mcz'!
>>>
>>> Shouldn't this be WebClient-Core-ar.92 without the .mcz?
>>
>> Yes, it should, but without the .mcz you'd end up loading
>> WebClient-Core-ar.921.mcz by accident, were there such a version.
>>
>> frank
>
>
> Really? I thought we fixed the matching from stupid strings to something more intelligent a while ago?

> :)
>
> I'm working on an InstallerGitHub that will pull in WebClient as
> needed: that means there'll be two things that load WebClient in
> lazily, requiring some coordination... or pulling out the common
> piece. I'm not quite sure how to do that since that introduces another
> dependency or two between things.
>
> And adding WebClient to Core seems... counterproductive!
>
> frank
>
> On 25 February 2013 22:50, Chris Muller <[hidden email]> wrote:
>> I'm glad you left it a hard-coded version (92) so that it will stay stable.
>>
>>
>> On Mon, Feb 25, 2013 at 3:27 PM,  <[hidden email]> wrote:
>>> Frank Shearar uploaded a new version of SMLoader to project The Trunk:
>>> http://source.squeak.org/trunk/SMLoader-fbs.78.mcz
>>>
>>> ==================== Summary ====================
>>>
>>> Name: SMLoader-fbs.78
>>> Author: fbs
>>> Time: 25 February 2013, 9:27:11.959 pm
>>> UUID: cd8023fe-60a1-4dd9-a7dd-60e8ce7f4ac5
>>> Ancestors: SMLoader-cmm.77
>>>
>>> When installing WebClient, use the most recent version.
>>>
>>> =============== Diff against SMLoader-cmm.77 ===============
>>>
>>> Item was changed:
>>>   SystemOrganization addCategory: #SMLoader!
>>> - SystemOrganization addCategory: #'SMLoader-Models'!
>>>
>>> Item was changed:
>>>   ----- Method: SMClient class>>installWebClient (in category 'private') -----
>>>   installWebClient
>>> +       Installer ss
>>> +               project: 'WebClient';
>>> +               install: 'WebClient-Core-ar.92.mcz'!
>>> -       (Installer repository: 'http://www.squeaksource.com/WebClient')
>>> -               install: 'WebClient-Core-ar.83'!
>>>
>>>
>>

> On Tue, Feb 26, 2013 at 1:21 AM, Frank Shearar <[hidden email]> wrote:
>> :)
>>
>> I'm working on an InstallerGitHub that will pull in WebClient as
>> needed: that means there'll be two things that load WebClient in
>> lazily, requiring some coordination... or pulling out the common
>> piece. I'm not quite sure how to do that since that introduces another
>> dependency or two between things.
>>
>> And adding WebClient to Core seems... counterproductive!
>>
>> frank
>>
>> On 25 February 2013 22:50, Chris Muller <[hidden email]> wrote:
>>> I'm glad you left it a hard-coded version (92) so that it will stay stable.
>>>
>>>
>>> On Mon, Feb 25, 2013 at 3:27 PM,  <[hidden email]> wrote:
>>>> Frank Shearar uploaded a new version of SMLoader to project The Trunk:
>>>> http://source.squeak.org/trunk/SMLoader-fbs.78.mcz
>>>>
>>>> ==================== Summary ====================
>>>>
>>>> Name: SMLoader-fbs.78
>>>> Author: fbs
>>>> Time: 25 February 2013, 9:27:11.959 pm
>>>> UUID: cd8023fe-60a1-4dd9-a7dd-60e8ce7f4ac5
>>>> Ancestors: SMLoader-cmm.77
>>>>
>>>> When installing WebClient, use the most recent version.
>>>>
>>>> =============== Diff against SMLoader-cmm.77 ===============
>>>>
>>>> Item was changed:
>>>>   SystemOrganization addCategory: #SMLoader!
>>>> - SystemOrganization addCategory: #'SMLoader-Models'!
>>>>
>>>> Item was changed:
>>>>   ----- Method: SMClient class>>installWebClient (in category 'private') -----
>>>>   installWebClient
>>>> +       Installer ss
>>>> +               project: 'WebClient';
>>>> +               install: 'WebClient-Core-ar.92.mcz'!
>>>> -       (Installer repository: 'http://www.squeaksource.com/WebClient')
>>>> -               install: 'WebClient-Core-ar.83'!
>>>>
>>>>
>>>

> On 26 February 2013 19:13, Chris Muller <[hidden email]> wrote:
>> What I did in my MaInstaller was to have a #merge: function which
>> would normally be used all the time (instead of #install:).  So
>> whichever one was more up-to-date would be loaded or, even if they
>> were branches of each other it would still work too as long as there
>> were no method-level conflicts.  (If there are method-level conflicts
>> then the interactive Merge dialog is presented where the user can make
>> a choice and proceed with the rest of the install by clicking Merge).
>>
>> If something isn't installed at yet, #merge: behaves exactly like #install:.
>
> OK, but that's not what I'm getting at. It's not actually a problem
> yet because this thing's not finished, but my point is that we'd have
> two things that lazily pulled in WebClient... and if SM simply had to
> use version N or later, and InstallerGitHub pulled in version N-1
> because you ran it before you opened the catalog, Bad Things might
> well happen. Especially so since "is WebClient installed?" is answered
> by Smalltalk at: #WebClient (really, this should be self class
> environment at: #WebClient). SM goes "oh cool, it's already loaded, so
> I'll just invoke this new behaviour OH NO MNU"

> On 26 February 2013 19:13, Chris Muller <[hidden email]> wrote:
>> What I did in my MaInstaller was to have a #merge: function which
>> would normally be used all the time (instead of #install:).  So
>> whichever one was more up-to-date would be loaded or, even if they
>> were branches of each other it would still work too as long as there
>> were no method-level conflicts.  (If there are method-level conflicts
>> then the interactive Merge dialog is presented where the user can make
>> a choice and proceed with the rest of the install by clicking Merge).
>>
>> If something isn't installed at yet, #merge: behaves exactly like #install:.
>
> OK, but that's not what I'm getting at. It's not actually a problem
> yet because this thing's not finished, but my point is that we'd have
> two things that lazily pulled in WebClient... and if SM simply had to
> use version N or later, and InstallerGitHub pulled in version N-1
> because you ran it before you opened the catalog, Bad Things might
> well happen. Especially so since "is WebClient installed?" is answered
> by Smalltalk at: #WebClient (really, this should be self class
> environment at: #WebClient). SM goes "oh cool, it's already loaded, so
> I'll just invoke this new behaviour OH NO MNU"

> On Tue, Feb 26, 2013 at 2:38 PM, Frank Shearar <[hidden email]> wrote:
>> On 26 February 2013 19:13, Chris Muller <[hidden email]> wrote:
>>> What I did in my MaInstaller was to have a #merge: function which
>>> would normally be used all the time (instead of #install:).  So
>>> whichever one was more up-to-date would be loaded or, even if they
>>> were branches of each other it would still work too as long as there
>>> were no method-level conflicts.  (If there are method-level conflicts
>>> then the interactive Merge dialog is presented where the user can make
>>> a choice and proceed with the rest of the install by clicking Merge).
>>>
>>> If something isn't installed at yet, #merge: behaves exactly like #install:.
>>
>> OK, but that's not what I'm getting at. It's not actually a problem
>> yet because this thing's not finished, but my point is that we'd have
>> two things that lazily pulled in WebClient... and if SM simply had to
>> use version N or later, and InstallerGitHub pulled in version N-1
>> because you ran it before you opened the catalog, Bad Things might
>> well happen. Especially so since "is WebClient installed?" is answered
>> by Smalltalk at: #WebClient (really, this should be self class
>> environment at: #WebClient). SM goes "oh cool, it's already loaded, so
>> I'll just invoke this new behaviour OH NO MNU"
>
> Hmm, so it seems to me the unconditional merge idea still solves all
> cases including that one doesn't it?  If SM was second, it would merge
> version N prior to its work.  If InstallerGitHub was second, its merge
> of N-1 would have no effect since it was already in the ancestry..
>
> In any case, as you said, this hardly seems like an issue because, if
> it were, the energy to rectify it wouldn't be to make them operate on
> different verisions but on the same, (most-recent) version.