On Tue, Aug 22, 2017 at 05:55:22PM +0200, Guillermo Polito wrote: > I don't think so... Build number only works from the CI. This means > that if you build your image in your machine you'll have no build > number and then this piece of code will not work neither. Thus maybe > we need to find a separate solution for it?
Good point. I guess that #registerToolClassNamed: was introduced around image #50558 and didn't exist in earlier versions, so the test could be replaced with: SessionManager default respondsTo: #registerToolClassNamed: I'll see if I can download an older Pharo 5 VM and image and check (pre 50558). Thanks, Alistair > On 8/22/17, Alistair Grant <[email protected]> wrote: > > Hi Everyone, > > > > I'm attempting to update to the latest Pharo7 image (918efe2, 32 bit), > > but am getting an error while loading OSsubprocess. > > > > The error occurs in FFISharedPool class>>initialize: > > > > > > initialize > > self flag: #removeIt. > > "This IF will be eliminated soon...only here temporary" > > (SystemVersion current highestUpdate >= 50558) > > ifTrue: [ > > SessionManager default registerToolClassNamed: self name > > ] > > ifFalse: [ > > Smalltalk addToStartUpList: self. > > ] > > > > > > because #highestUpdate is nil. > > > > > > The top of the stack is: > > > > UndefinedObject(Object)>>doesNotUnderstand: #'>=' > > FFISharedPool class>>initialize > > MCMethodDefinition>>postloadOver: > > [ :each | each postloadOver: (self obsoletionFor: each) ] in > > MCPackageLoader>>basicLoadDefinitions in Block: [ :each | each postloadOver: > > (self obsoletionFor: ...etc... > > > > > > Based on the method above, it looks like highestUpdate used to be set > > based on the image number, and has presumably broken during the move to > > Pharo 7 and git. > > > > Earlier Pharo 7 images had it set to 0, which doesn't sound right. > > > > Since a build number was recently introduced (fogbugz 20321), should the > > highestUpdate be set to 70000+buildNumber? > > > > E.g. this image has an identifier of: > > > > Pharo 7.0 > > Build information: > > alpha.build.56.sha.918efe28fa3e6ad485b059dbf4ce675fd4086cf6 > > > > So highestUpdate would be 70056. > > > > > > Thanks! > > Alistair > > > > > > > -- > > > > > Guille Polito > > > Research Engineer > > French National Center for Scientific Research - *http://www.cnrs.fr* > <http://www.cnrs.fr> > > > > *Web:* *http://guillep.github.io* <http://guillep.github.io> > > *Phone: *+33 06 52 70 66 13 >
