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,  <commits at source.squeak.org> 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).!
> >
> >