3.9-7061 showstopper bug - Mantis #5231

stephane ducasse stephane.ducasse at gmail.com
Sat Oct 14 15:03:25 UTC 2006


Thanks Scott.
from the timestamp I think that this is the notification mechanism  
from nathanael.
We will see if we can introduce it in 3.9

Stef

On 14 oct. 06, at 14:33, Scott Wallace wrote:

> The bug was not only present in 3.8, it was already present in 3.7.
>
> It arrived in update "5706KCP170CompilerPrtclRefactor", in a method  
> timestamped "NS 1/28/2004".  So, amazingly, the community has been  
> tolerating this showstopping bug in its "stable" releases for more  
> than 2.5 years.
>
> I personally was unwilling to live with the bug for even a day,  
> once I encountered it;  so for me, as for Tim, this was a  
> showstopper, no irony intended.   Not only can the .changes file  
> get quickly filled with arbitrarily large amounts of arbitrary  
> text, but the unformatted logging of arbitrary text causes havoc  
> with attempts of the ChangeList browser to parse chunks from  
> the .changes file.  So to me this was a completely intolerable bug.
>
> I quickly tracked down the cause to the "Kernel Cleaning Project,"  
> in the change-set mentioned above.  I couldn't immediately devise  
> an obvious, elegant fix, but I did settle on a workaround that did  
> the job, however unappetizingly.  Since that time there have been  
> two further advances on the method, one for TinLizzie (for Tweak  
> compatibility) and the latest for OLPC.
>
> I attach the fileout that constitutes the latest OLPC incarnation  
> of this method, which is more robust than the Squeakland version.   
> I don't know how this would integrate with 3.9 -- Marcus must have  
> had good reasons for not porting it 3.9 -- but I offer it FWIW.
>
> Cheers,
>
>  -- Scott
>
>
>
> <1025evalFix-sw.cs.gz>
>
>
> On Oct 13, 2006, at 12:21 AM, Marcus Denker wrote:
>
>>
>> On 13.10.2006, at 01:38, tim Rowledge wrote:
>>
>>> I consider this to be a showstopper:-
>>>
>>> open a workspace
>>> type in some random stuff
>>> add a line
>>> 	'3+4'
>>> and print it on the '3+4'
>>> Now use menu->changes->recent log file... to see what got put in  
>>> the changes log.
>>>
>>> Whoops! Everything in the workspace up to the code printit'd is  
>>> logged.
>>> If you put some evaluable in the middle of the workspace and  
>>> printit, again you get everything up to the printit.
>>>
>>> We can't let this go out for general use - it will fill the  
>>> changes log much faster than anyone could ever expect and would  
>>> completely break the logic of recovering your state by executing  
>>> the recent changes file since many things would get repeated  
>>> repeatedly. Not to mention, the entire text in the recovered file  
>>> up to the chunk being evaluated would get logged *again*.
>>>
>>
>> This does not seem to be 3.9 specific, as there was a fix for that  
>> in squeakland...
>> I did not merge 3 of 150 changesets, and this is one of those:
>>
>> http://bugs.impara.de/view.php?id=3331
>>
>> Change Set: twoFixes-sw
>> Date: 12 November 2005
>> Author: Scott Wallace
>>
>> Fixes the bug that logged the entire contents of a text pane up  
>> through the end of the current selection, rather than only the  
>> current selection, when logging a do-it to the changes.
>>
>> ------
>>
>> If this is in 3.8, too, then by definition it is not a show  
>> stopping bug. (As the show was not stopped with 3.8). So this  
>> would need to be checked.
>>
>>      Marcus
>>
>
>




More information about the Squeak-dev mailing list