[squeak-dev] Re: Closures in Trunk

Bert Freudenberg bert at freudenbergs.de
Wed Aug 12 19:34:36 UTC 2009


I just had another look:

	MCMcdReader versionFromFile: 'Kernel-ar.188(dtl.187).mcd'

Indeed there is no preamble in the patch. Now, why ...

*smacks head*

The problem is the version of Monticello on the source server, which  
creates the diffs but does not know preambles.

- Bert -

On 12.08.2009, at 18:53, Ken Causey wrote:

> So back to the original problem with Kernel-ar.188...  I hadn't  
> updated
> in a few days (ok, maybe more than a week) and finally did so today
> trying out the 3.10-6 unix vm and ran into this bug.  Having a  
> terrible
> memory I had to search through the mailing list to figure out what to
> do.  Are we just going to leave this issue at the current 'requires a
> workaround' status?  I hope not.
>
> Ken
>
> On Tue, 2009-07-21 at 08:43 -0300, Bert Freudenberg wrote:
>> On 21.07.2009, at 00:54, Andreas Raab wrote:
>>
>>> Bert Freudenberg wrote:
>>>> IIRC, preambles are not handled specifically by DiffyVersions. So
>>>> when a preamble is requested of the diffy version and it has none,
>>>> then it would have to look in its base version. Not sure if this is
>>>> working, it probably has never been tested before. Alternatively,
>>>> the server could be modified to always include preambles in mcds.
>>>
>>> I think the issue might be related to the apparent problem that
>>> preambles and postscripts do not appear to be included in the
>>> sources, but only stored in binary form. This seems badly broken to
>>> say the least - if the loader ever needs to fall back to source
>>> parsing, both preambles and postscripts would be completely ignored.
>>> I could see how that could cause problems along the way.
>>
>>
>> Indeed, that is a problem. I never looked how they were actually
>> implemented ... maybe newer MC versions do better?
>>
>> - Bert -
>>
>>
>>
>>
>






More information about the Squeak-dev mailing list