Bug: Metacello caches some version files with 4 bytes per character

> Gofer new
> package: 'GsUpgrader-Core';
> url: 'http://ss3.gemtalksystems.com/ss/gsUpgrader';
> load.
> (Smalltalk at: #GsUpgrader)
> upgradeGrease;
> upgradeGLASS1.
> (Smalltalk at: #GsDeployer)
> deploy: [
> Metacello new
> baseline: 'Seaside3';
> repository: 'github://GsDevKit/Seaside31:v3.1.3.1-gs/repository';
> onLock: [ :ex | ex honor ];
> load: #('Core' 'Seaside-Development' 'Seaside-Adaptors-FastCGI').
> Metacello new
> configuration: 'Magritte3';
> version: #'release3.1';
> repository: 'http://www.smalltalkhub.com/mc/Magritte/Magritte3/main';
> onConflict: [ :ex :existingRegistration :newRegistration |
> Transcript
> show:
> 'Conflict between existing: ' , existingRegistration className ,
> ' ' , existingRegistration printString
> , Character cr asString , '  and new: ' , newRegistration
> className , ' ' , newRegistration printString
> , Character cr asString , '  resolved in favor of existing'.
> ex disallow ];
> cacheRepository: '/tmp/somewhere';
> load: 'Magritte-Seaside' ]

>
> Dale
>
>
> On 01/28/2015 04:02 AM, Pieter Nagel wrote:
>> Gofer new
>> package: 'GsUpgrader-Core';
>> url: 'http://ss3.gemtalksystems.com/ss/gsUpgrader';
>> load.
>> (Smalltalk at: #GsUpgrader)
>> upgradeGrease;
>> upgradeGLASS1.
>> (Smalltalk at: #GsDeployer)
>> deploy: [
>> Metacello new
>> baseline: 'Seaside3';
>> repository: 'github://GsDevKit/Seaside31:v3.1.3.1-gs/repository';
>> onLock: [ :ex | ex honor ];
>> load: #('Core' 'Seaside-Development' 'Seaside-Adaptors-FastCGI').
>> Metacello new
>> configuration: 'Magritte3';
>> version: #'release3.1';
>> repository:
>> 'http://www.smalltalkhub.com/mc/Magritte/Magritte3/main';
>> onConflict: [ :ex :existingRegistration :newRegistration |
>> Transcript
>> show:
>> 'Conflict between existing: ' , existingRegistration className
>> ,
>> ' ' , existingRegistration printString
>> , Character cr asString , '  and new: ' , newRegistration
>> className , ' ' , newRegistration printString
>> , Character cr asString , '  resolved in favor of existing'.
>> ex disallow ];
>> cacheRepository: '/tmp/somewhere';
>> load: 'Magritte-Seaside' ]
>
> --
> You received this message because you are subscribed to the Google Groups
> "Metacello" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to [hidden email].
> For more options, visit https://groups.google.com/d/optout.
>
>

> We are unable to load Magritte3 over Seaside3.1 when Metacello is
> simultaneously writing to a cacheRepository.
>
> The problem seems to be that when Metacello caches, it also uses the mcz
> it wrote to load from, and in the case of JQuery-Core-JohanBrichau.132.mcz
> which it fetched from github://GsDevKit/Seaside31:v3.1.3.1-gs/repository,
> it writes a version file to the mcz that has 4 bytes per character - which
> it then later of course can not parse.
>
> In order to reproduce the bug, log in to an extent0.seaside.dbf under
> GemStone64 3.1.05, create a directory /tmp/somewhere, and execute the
> following:
>
> Gofer new
> package: 'GsUpgrader-Core';
> url: 'http://ss3.gemtalksystems.com/ss/gsUpgrader';
> load.
> (Smalltalk at: #GsUpgrader)
> upgradeGrease;
> upgradeGLASS1.
> (Smalltalk at: #GsDeployer)
> deploy: [
> Metacello new
> baseline: 'Seaside3';
> repository: 'github://GsDevKit/Seaside31:v3.1.3.1-gs/repository';
> onLock: [ :ex | ex honor ];
> load: #('Core' 'Seaside-Development' 'Seaside-Adaptors-FastCGI').
> Metacello new
> configuration: 'Magritte3';
> version: #'release3.1';
> repository: 'http://www.smalltalkhub.com/mc/Magritte/Magritte3/main';
> onConflict: [ :ex :existingRegistration :newRegistration |
> Transcript
> show:
> 'Conflict between existing: ' , existingRegistration className ,
> ' ' , existingRegistration printString
> , Character cr asString , '  and new: ' , newRegistration
> className , ' ' , newRegistration printString
> , Character cr asString , '  resolved in favor of existing'.
> ex disallow ];
> cacheRepository: '/tmp/somewhere';
> load: 'Magritte-Seaside' ]
>
>
>
>
>
>
>

>> a LookupError occurred (error 2021), reason:rtErrKeyNotFound, A
>> reference using the non-existent key #'id' was made into the dictionary
>> aDictionary( #'e'->#'s', #'m'->#'e', #'l'->#'d', '....
> Yup, that's the same error I got, I can drop in a the client tomorrow and
> mail you the stacktrace from there, but it seems you are looking at the
> same one I did.
>
> It seems that whatever code parses the version files gets confused by the
> extra zero bytes, and so a dictionary that should have contained mappings
> from #'version' to version and whatnot now just contains mappings between
> characters that were adjacent in that file, or something like that.
>
> If you go and unzip /tmp/somewhere/Query-Core-JohanBrichau.132.mcz as it
> was cached and look at the version file with a hex editor (or something
> that can show zero bytes), you'll see the problem clearly.
>
> I see at
> https://github.com/GsDevKit/Seaside31/blob/gs_master/repository/JQuery-Core.package/monticello.meta/version
> that the version file on GitHub has at least one non-ASCII character
> encoded as two bytes in the phrase "from Smalltalks 2010 in
> Concepci󮠤el Uruguay", but it doesn't seem to be UTF-8 and I'm not
> sure what encoding version files should nominally be in.
>
> My hunch is that something about that character caused a QuadByteString or
> the like to be used for the text of the version file, hence the 4 bytes
> per character.
>
>
>
>> Dale
>>
>>
>> On 01/28/2015 04:02 AM, Pieter Nagel wrote:
>>> Gofer new
>>> package: 'GsUpgrader-Core';
>>> url: 'http://ss3.gemtalksystems.com/ss/gsUpgrader';
>>> load.
>>> (Smalltalk at: #GsUpgrader)
>>> upgradeGrease;
>>> upgradeGLASS1.
>>> (Smalltalk at: #GsDeployer)
>>> deploy: [
>>> Metacello new
>>> baseline: 'Seaside3';
>>> repository: 'github://GsDevKit/Seaside31:v3.1.3.1-gs/repository';
>>> onLock: [ :ex | ex honor ];
>>> load: #('Core' 'Seaside-Development' 'Seaside-Adaptors-FastCGI').
>>> Metacello new
>>> configuration: 'Magritte3';
>>> version: #'release3.1';
>>> repository:
>>> 'http://www.smalltalkhub.com/mc/Magritte/Magritte3/main';
>>> onConflict: [ :ex :existingRegistration :newRegistration |
>>> Transcript
>>> show:
>>> 'Conflict between existing: ' , existingRegistration className
>>> ,
>>> ' ' , existingRegistration printString
>>> , Character cr asString , '  and new: ' , newRegistration
>>> className , ' ' , newRegistration printString
>>> , Character cr asString , '  resolved in favor of existing'.
>>> ex disallow ];
>>> cacheRepository: '/tmp/somewhere';
>>> load: 'Magritte-Seaside' ]
>> --
>> You received this message because you are subscribed to the Google Groups
>> "Metacello" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to [hidden email].
>> For more options, visit https://groups.google.com/d/optout.
>>
>>
>