I have a problem with 11126

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

I have a problem with 11126

Stéphane Ducasse
hi igor


up to 11125 it works well and after when I was trying to do the last two cs using MC

in 11125 I get

new
        ^ self trailerClass basicNew initialize
trailerClass
        ^ TrailerClass ifNil: [ OldMethodTrailer ]

in 11126 when I try to load these packages.

I get  CompiledMethodTrailer new
                         does understand trailerClass






Stef (now I have to run downtown with my boys and I will come back in 2hours or so).

CompiledMethodTrailer class(Object)>>doesNotUnderstand: #trailerClass
CompiledMethodTrailer class>>new
CompiledMethod>>trailer
CompiledMethod>>getSourceFor:in:
Metaclass(Behavior)>>sourceCodeAt:ifAbsent:
Metaclass(Behavior)>>ultimateSourceCodeAt:ifAbsent:
Browser(CodeHolder)>>sourceStringPrettifiedAndDiffed
Browser>>selectedMessage
Browser>>contents
PluggableTextMorph>>getText
PluggableTextMorph>>update:
[] in Browser(Object)>>changed:
DependentsArray>>do:
Browser(Object)>>changed:
Browser(Object)>>contentsChanged
Browser(CodeHolder)>>contentsChanged
Browser>>messageListIndex:
PluggableListMorph>>changeModelSelection:
PluggableListMorph>>mouseUp:
PluggableListMorph(Morph)>>handleMouseUp:
MouseButtonEvent>>sentTo:
PluggableListMorph(Morph)>>handleEvent:
MorphicEventDispatcher>>dispatchDefault:with:
MorphicEventDispatcher>>dispatchEvent:with:
PluggableListMorph(Morph)>>processEvent:using:
PluggableListMorph(Morph)>>processEvent:
PluggableListMorph>>handleFocusEvent:
[] in HandMorph>>sendFocusEvent:to:clear:
[] in PasteUpMorph>>becomeActiveDuring:
BlockClosure>>on:do:
PasteUpMorph>>becomeActiveDuring:
HandMorph>>sendFocusEvent:to:clear:
HandMorph>>sendEvent:focus:clear:
HandMorph>>sendMouseEvent:
HandMorph>>handleEvent:
HandMorph>>processEvents
[] in WorldState>>doOneCycleNowFor:
Array(SequenceableCollection)>>do:
WorldState>>handsDo:
_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Collections-Abstract-StephaneDucasse.49.mcz (71K) Download Attachment
Kernel-StephaneDucasse.531.mcz (966K) Download Attachment
ScriptLoader11-StephaneDucasse.135.mcz (154K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: I have a problem with 11126

Igor Stasenko
2009/12/30 Stéphane Ducasse <[hidden email]>:

> hi igor
>
>
> up to 11125 it works well and after when I was trying to do the last two cs using MC
>
> in 11125 I get
>
> new
>        ^ self trailerClass basicNew initialize
> trailerClass
>        ^ TrailerClass ifNil: [ OldMethodTrailer ]
>
> in 11126 when I try to load these packages.
>
> I get  CompiledMethodTrailer new
>                         does understand trailerClass
>
>

You seem stepped on same trap as Andreas did.  :)
MC minds own method removal order,
and removes #trailerClass before #new,
which then leads to DNU.

Split cleanup into two stages:
- in first update, remove just #new
- in second , remove #trailerClass and the rest of stuff

so, MC will have no chances to interfere with removal order.

--
Best regards,
Igor Stasenko AKA sig.

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: I have a problem with 11126

Stéphane Ducasse
I'm back from christmas shpping mess.
ok I will do it.


On Dec 30, 2009, at 5:20 PM, Igor Stasenko wrote:

> 2009/12/30 Stéphane Ducasse <[hidden email]>:
>> hi igor
>>
>>
>> up to 11125 it works well and after when I was trying to do the last two cs using MC
>>
>> in 11125 I get
>>
>> new
>>        ^ self trailerClass basicNew initialize
>> trailerClass
>>        ^ TrailerClass ifNil: [ OldMethodTrailer ]
>>
>> in 11126 when I try to load these packages.
>>
>> I get  CompiledMethodTrailer new
>>                         does understand trailerClass
>>
>>
>
> You seem stepped on same trap as Andreas did.  :)
> MC minds own method removal order,
> and removes #trailerClass before #new,
> which then leads to DNU.
>
> Split cleanup into two stages:
> - in first update, remove just #new
> - in second , remove #trailerClass and the rest of stuff
>
> so, MC will have no chances to interfere with removal order.
>
> --
> Best regards,
> Igor Stasenko AKA sig.
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: I have a problem with 11126

Stéphane Ducasse
Igor I push the latest bit out.
I did not remove trailerClass because the classVariable was still there.
Let me know if it should be there or not (not time to check - wife shouting for diner :)

Stef
On Dec 30, 2009, at 6:10 PM, Stéphane Ducasse wrote:

> I'm back from christmas shpping mess.
> ok I will do it.
>
>
> On Dec 30, 2009, at 5:20 PM, Igor Stasenko wrote:
>
>> 2009/12/30 Stéphane Ducasse <[hidden email]>:
>>> hi igor
>>>
>>>
>>> up to 11125 it works well and after when I was trying to do the last two cs using MC
>>>
>>> in 11125 I get
>>>
>>> new
>>>       ^ self trailerClass basicNew initialize
>>> trailerClass
>>>       ^ TrailerClass ifNil: [ OldMethodTrailer ]
>>>
>>> in 11126 when I try to load these packages.
>>>
>>> I get  CompiledMethodTrailer new
>>>                        does understand trailerClass
>>>
>>>
>>
>> You seem stepped on same trap as Andreas did.  :)
>> MC minds own method removal order,
>> and removes #trailerClass before #new,
>> which then leads to DNU.
>>
>> Split cleanup into two stages:
>> - in first update, remove just #new
>> - in second , remove #trailerClass and the rest of stuff
>>
>> so, MC will have no chances to interfere with removal order.
>>
>> --
>> Best regards,
>> Igor Stasenko AKA sig.
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: I have a problem with 11126

Igor Stasenko
2009/12/30 Stéphane Ducasse <[hidden email]>:
> Igor I push the latest bit out.
> I did not remove trailerClass because the classVariable was still there.
> Let me know if it should be there or not (not time to check - wife shouting for diner :)
>
Once you remove #new, you can remove everything else in any order you
prefer, including trailerClass & classvar.

> Stef
> On Dec 30, 2009, at 6:10 PM, Stéphane Ducasse wrote:
>
>> I'm back from christmas shpping mess.
>> ok I will do it.
>>
>>
>> On Dec 30, 2009, at 5:20 PM, Igor Stasenko wrote:
>>
>>> 2009/12/30 Stéphane Ducasse <[hidden email]>:
>>>> hi igor
>>>>
>>>>
>>>> up to 11125 it works well and after when I was trying to do the last two cs using MC
>>>>
>>>> in 11125 I get
>>>>
>>>> new
>>>>       ^ self trailerClass basicNew initialize
>>>> trailerClass
>>>>       ^ TrailerClass ifNil: [ OldMethodTrailer ]
>>>>
>>>> in 11126 when I try to load these packages.
>>>>
>>>> I get  CompiledMethodTrailer new
>>>>                        does understand trailerClass
>>>>
>>>>
>>>
>>> You seem stepped on same trap as Andreas did.  :)
>>> MC minds own method removal order,
>>> and removes #trailerClass before #new,
>>> which then leads to DNU.
>>>
>>> Split cleanup into two stages:
>>> - in first update, remove just #new
>>> - in second , remove #trailerClass and the rest of stuff
>>>
>>> so, MC will have no chances to interfere with removal order.
>>>
>>> --
>>> Best regards,
>>> Igor Stasenko AKA sig.
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>



--
Best regards,
Igor Stasenko AKA sig.

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: I have a problem with 11126

Stéphane Ducasse
Ok this is strange that the classVar was not removed in the cleanup.
Anyway I will do it. this is cool to have such part of the system cleaned that way.

Stef




_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project