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).! |
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 |
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).! > > |
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).! > > > > |
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).! >> > >> > |
Free forum by Nabble | Edit this page |