Squeak 4.6: MonticelloConfigurations-dtl.136.mcz

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

Squeak 4.6: MonticelloConfigurations-dtl.136.mcz

commits-2
David T. Lewis uploaded a new version of MonticelloConfigurations to project Squeak 4.6:
http://source.squeak.org/squeak46/MonticelloConfigurations-dtl.136.mcz

==================== Summary ====================

Name: MonticelloConfigurations-dtl.136
Author: dtl
Time: 5 July 2015, 1:05:12.418 pm
UUID: ce452136-fc06-4d15-b41c-efd42ac6c0f8
Ancestors: MonticelloConfigurations-dtl.135

Only the default MCMcmUpdater should update the system version.

This fixes a condition in which loading a "head" package from SqueakMap using an update stream overwrote the system version.

=============== Diff against MonticelloConfigurations-dtl.135 ===============

Item was changed:
  ----- Method: MCMcmUpdater>>updateFrom: (in category 'updating') -----
  updateFrom: url
+ "Update the image by loading all pending updates from the server. If this is
+ the default updater for the system, update the system version when complete."
- "Update the image by loading all pending updates from the server."
  | config |
  "Flush all caches. If a previous download failed this is often helpful"
  MCFileBasedRepository flushAllCaches.
  config := self updateFromRepositories: { url }.
  config ifNil: [^self inform: 'Unable to retrieve updates from remote repository.' translated].
+ MCMcmUpdater default == self
+ ifTrue: [ config setSystemVersion.
+ self inform: ('Update completed.
+ Current update number: ' translated, SystemVersion current highestUpdate).]
+ ifFalse: [self inform: 'Update completed.']
+ !
- config setSystemVersion.
- self inform: ('Update completed.
- Current update number: ' translated, SystemVersion current highestUpdate).!


Reply | Threaded
Open this post in threaded view
|

Re: Squeak 4.6: MonticelloConfigurations-dtl.136.mcz

Chris Muller-3
On Sun, Jul 5, 2015 at 12:05 PM,  <[hidden email]> wrote:

> David T. Lewis uploaded a new version of MonticelloConfigurations to project Squeak 4.6:
> http://source.squeak.org/squeak46/MonticelloConfigurations-dtl.136.mcz
>
> ==================== Summary ====================
>
> Name: MonticelloConfigurations-dtl.136
> Author: dtl
> Time: 5 July 2015, 1:05:12.418 pm
> UUID: ce452136-fc06-4d15-b41c-efd42ac6c0f8
> Ancestors: MonticelloConfigurations-dtl.135
>
> Only the default MCMcmUpdater should update the system version.
>
> This fixes a condition in which loading a "head" package from SqueakMap using an update stream overwrote the system version.
>
> =============== Diff against MonticelloConfigurations-dtl.135 ===============
>
> Item was changed:
>   ----- Method: MCMcmUpdater>>updateFrom: (in category 'updating') -----
>   updateFrom: url
> +       "Update the image by loading all pending updates from the server. If this is
> +       the default updater for the system, update the system version when complete."
> -       "Update the image by loading all pending updates from the server."
>         | config |
>         "Flush all caches. If a previous download failed this is often helpful"
>         MCFileBasedRepository flushAllCaches.
>         config := self updateFromRepositories: { url }.
>         config ifNil: [^self inform: 'Unable to retrieve updates from remote repository.' translated].
> +       MCMcmUpdater default == self
> +               ifTrue: [       config setSystemVersion.
> +                       self inform: ('Update completed.
> + Current update number: ' translated, SystemVersion current highestUpdate).]
> +               ifFalse: [self inform: 'Update completed.']
> +       !
> -       config setSystemVersion.
> -       self inform: ('Update completed.
> - Current update number: ' translated, SystemVersion current highestUpdate).!
>
>



ee.png (85K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: Squeak 4.6: MonticelloConfigurations-dtl.136.mcz

Chris Muller-3
In reply to this post by commits-2
I want to build RC2 with All-In-One but am suddenly getting an
emergency evaluator when trying to update my trunk image.  Is it
related to this?  It's the last thing on the Progress bar ..


On Sun, Jul 5, 2015 at 12:05 PM,  <[hidden email]> wrote:

> David T. Lewis uploaded a new version of MonticelloConfigurations to project Squeak 4.6:
> http://source.squeak.org/squeak46/MonticelloConfigurations-dtl.136.mcz
>
> ==================== Summary ====================
>
> Name: MonticelloConfigurations-dtl.136
> Author: dtl
> Time: 5 July 2015, 1:05:12.418 pm
> UUID: ce452136-fc06-4d15-b41c-efd42ac6c0f8
> Ancestors: MonticelloConfigurations-dtl.135
>
> Only the default MCMcmUpdater should update the system version.
>
> This fixes a condition in which loading a "head" package from SqueakMap using an update stream overwrote the system version.
>
> =============== Diff against MonticelloConfigurations-dtl.135 ===============
>
> Item was changed:
>   ----- Method: MCMcmUpdater>>updateFrom: (in category 'updating') -----
>   updateFrom: url
> +       "Update the image by loading all pending updates from the server. If this is
> +       the default updater for the system, update the system version when complete."
> -       "Update the image by loading all pending updates from the server."
>         | config |
>         "Flush all caches. If a previous download failed this is often helpful"
>         MCFileBasedRepository flushAllCaches.
>         config := self updateFromRepositories: { url }.
>         config ifNil: [^self inform: 'Unable to retrieve updates from remote repository.' translated].
> +       MCMcmUpdater default == self
> +               ifTrue: [       config setSystemVersion.
> +                       self inform: ('Update completed.
> + Current update number: ' translated, SystemVersion current highestUpdate).]
> +               ifFalse: [self inform: 'Update completed.']
> +       !
> -       config setSystemVersion.
> -       self inform: ('Update completed.
> - Current update number: ' translated, SystemVersion current highestUpdate).!
>
>

Reply | Threaded
Open this post in threaded view
|

Re: Squeak 4.6: MonticelloConfigurations-dtl.136.mcz

David T. Lewis
On Sun, Jul 05, 2015 at 10:47:00PM -0500, Chris Muller wrote:
> I want to build RC2 with All-In-One but am suddenly getting an
> emergency evaluator when trying to update my trunk image.  Is it
> related to this?  It's the last thing on the Progress bar ..

No, I don't think it is related. This change is just to prevent a
non-default update stream from changing the system version. For example,
I have the SqueakMap "head" entries for OSProcess and CommandShell
set to use the update maps in those repositories. The error (fixed
in MonticelloConfigurations-dtl.136) was that those update processes
were stomping on the system version that is supposed to apply to
the default (trunk or squeak46) update stream. Thus, if you loaded
OSProcess head from SqueakMap, the system version was getting set to
a completely bogus value. The change here is to make sure that the
system version is updated if and only if the updater is for the
default update stream.

Dave

>
>
> On Sun, Jul 5, 2015 at 12:05 PM,  <[hidden email]> wrote:
> > David T. Lewis uploaded a new version of MonticelloConfigurations to project Squeak 4.6:
> > http://source.squeak.org/squeak46/MonticelloConfigurations-dtl.136.mcz
> >
> > ==================== Summary ====================
> >
> > Name: MonticelloConfigurations-dtl.136
> > Author: dtl
> > Time: 5 July 2015, 1:05:12.418 pm
> > UUID: ce452136-fc06-4d15-b41c-efd42ac6c0f8
> > Ancestors: MonticelloConfigurations-dtl.135
> >
> > Only the default MCMcmUpdater should update the system version.
> >
> > This fixes a condition in which loading a "head" package from SqueakMap using an update stream overwrote the system version.
> >
> > =============== Diff against MonticelloConfigurations-dtl.135 ===============
> >
> > Item was changed:
> >   ----- Method: MCMcmUpdater>>updateFrom: (in category 'updating') -----
> >   updateFrom: url
> > +       "Update the image by loading all pending updates from the server. If this is
> > +       the default updater for the system, update the system version when complete."
> > -       "Update the image by loading all pending updates from the server."
> >         | config |
> >         "Flush all caches. If a previous download failed this is often helpful"
> >         MCFileBasedRepository flushAllCaches.
> >         config := self updateFromRepositories: { url }.
> >         config ifNil: [^self inform: 'Unable to retrieve updates from remote repository.' translated].
> > +       MCMcmUpdater default == self
> > +               ifTrue: [       config setSystemVersion.
> > +                       self inform: ('Update completed.
> > + Current update number: ' translated, SystemVersion current highestUpdate).]
> > +               ifFalse: [self inform: 'Update completed.']
> > +       !
> > -       config setSystemVersion.
> > -       self inform: ('Update completed.
> > - Current update number: ' translated, SystemVersion current highestUpdate).!
> >
> >

Reply | Threaded
Open this post in threaded view
|

Re: Squeak 4.6: MonticelloConfigurations-dtl.136.mcz

Chris Muller-4
Ah, that makes sense.  Good fix.

Maybe the problem is that Compiler needs to be moved up in the mcm.
Its too late I'll have to check tomorrow.

On Sun, Jul 5, 2015 at 11:02 PM, David T. Lewis <[hidden email]> wrote:

> On Sun, Jul 05, 2015 at 10:47:00PM -0500, Chris Muller wrote:
>> I want to build RC2 with All-In-One but am suddenly getting an
>> emergency evaluator when trying to update my trunk image.  Is it
>> related to this?  It's the last thing on the Progress bar ..
>
> No, I don't think it is related. This change is just to prevent a
> non-default update stream from changing the system version. For example,
> I have the SqueakMap "head" entries for OSProcess and CommandShell
> set to use the update maps in those repositories. The error (fixed
> in MonticelloConfigurations-dtl.136) was that those update processes
> were stomping on the system version that is supposed to apply to
> the default (trunk or squeak46) update stream. Thus, if you loaded
> OSProcess head from SqueakMap, the system version was getting set to
> a completely bogus value. The change here is to make sure that the
> system version is updated if and only if the updater is for the
> default update stream.
>
> Dave
>
>>
>>
>> On Sun, Jul 5, 2015 at 12:05 PM,  <[hidden email]> wrote:
>> > David T. Lewis uploaded a new version of MonticelloConfigurations to project Squeak 4.6:
>> > http://source.squeak.org/squeak46/MonticelloConfigurations-dtl.136.mcz
>> >
>> > ==================== Summary ====================
>> >
>> > Name: MonticelloConfigurations-dtl.136
>> > Author: dtl
>> > Time: 5 July 2015, 1:05:12.418 pm
>> > UUID: ce452136-fc06-4d15-b41c-efd42ac6c0f8
>> > Ancestors: MonticelloConfigurations-dtl.135
>> >
>> > Only the default MCMcmUpdater should update the system version.
>> >
>> > This fixes a condition in which loading a "head" package from SqueakMap using an update stream overwrote the system version.
>> >
>> > =============== Diff against MonticelloConfigurations-dtl.135 ===============
>> >
>> > Item was changed:
>> >   ----- Method: MCMcmUpdater>>updateFrom: (in category 'updating') -----
>> >   updateFrom: url
>> > +       "Update the image by loading all pending updates from the server. If this is
>> > +       the default updater for the system, update the system version when complete."
>> > -       "Update the image by loading all pending updates from the server."
>> >         | config |
>> >         "Flush all caches. If a previous download failed this is often helpful"
>> >         MCFileBasedRepository flushAllCaches.
>> >         config := self updateFromRepositories: { url }.
>> >         config ifNil: [^self inform: 'Unable to retrieve updates from remote repository.' translated].
>> > +       MCMcmUpdater default == self
>> > +               ifTrue: [       config setSystemVersion.
>> > +                       self inform: ('Update completed.
>> > + Current update number: ' translated, SystemVersion current highestUpdate).]
>> > +               ifFalse: [self inform: 'Update completed.']
>> > +       !
>> > -       config setSystemVersion.
>> > -       self inform: ('Update completed.
>> > - Current update number: ' translated, SystemVersion current highestUpdate).!
>> >
>> >