[Pharo-project] I have a problem with 11126
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:
2009/12/30 Stéphane Ducasse <stephane.ducasse@inria.fr>:
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.
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 <stephane.ducasse@inria.fr>:
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 Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
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 <stephane.ducasse@inria.fr>:
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 Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
2009/12/30 Stéphane Ducasse <stephane.ducasse@inria.fr>:
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 <stephane.ducasse@inria.fr>:
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 Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
-- Best regards, Igor Stasenko AKA sig.
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
participants (2)
-
Igor Stasenko -
Stéphane Ducasse