[Pharo-project] testInstanceSideMethodsWithNilKeyInLastLiteral is still red
Camille I integrated your second version of the issue but I still get testInstanceSideMethodsWithNilKeyInLastLiteral self assert: SystemNavigation new instanceSideMethodsWithNilKeyInLastLiteral isEmpty
On Jan 27, 2013, at 5:20 PM, Stéphane Ducasse <stephane.ducasse@inria.fr> wrote:
Camille I integrated your second version of the issue but I still get
testInstanceSideMethodsWithNilKeyInLastLiteral
self assert: SystemNavigation new instanceSideMethodsWithNilKeyInLastLiteral isEmpty
I think we need to recompile the offending methods once. Marcus
Did your run the script from issue 7169? It is this one: SystemNavigation new instanceSideMethodsWithNilKeyInLastLiteral do: [ :thisShouldNotBeAStringButAMethod | | method classAndSelector | classAndSelector := ' ' split: thisShouldNotBeAStringButAMethod. method := (Smalltalk at: classAndSelector first asSymbol) methodNamed: classAndSelector second asSymbol. method recompile. ]. But if I remember it introduces a conflict with Fuel in MetalevelTests. I'm checking. On 27 janv. 2013, at 17:19, Stéphane Ducasse wrote:
Camille I integrated your second version of the issue but I still get
testInstanceSideMethodsWithNilKeyInLastLiteral
self assert: SystemNavigation new instanceSideMethodsWithNilKeyInLastLiteral isEmpty
No, it's ok after running the script. On 27 janv. 2013, at 18:17, wam wrote:
Did your run the script from issue 7169? It is this one: SystemNavigation new instanceSideMethodsWithNilKeyInLastLiteral do: [ :thisShouldNotBeAStringButAMethod | | method classAndSelector | classAndSelector := ' ' split: thisShouldNotBeAStringButAMethod. method := (Smalltalk at: classAndSelector first asSymbol) methodNamed: classAndSelector second asSymbol. method recompile. ]. But if I remember it introduces a conflict with Fuel in MetalevelTests. I'm checking.
On 27 janv. 2013, at 17:19, Stéphane Ducasse wrote:
Camille I integrated your second version of the issue but I still get
testInstanceSideMethodsWithNilKeyInLastLiteral
self assert: SystemNavigation new instanceSideMethodsWithNilKeyInLastLiteral isEmpty
Ok excellent I forgot to run the script! Thanks I will do that in the next update. Stef On Jan 27, 2013, at 2:24 PM, wam wrote:
No, it's ok after running the script.
On 27 janv. 2013, at 18:17, wam wrote:
Did your run the script from issue 7169? It is this one: SystemNavigation new instanceSideMethodsWithNilKeyInLastLiteral do: [ :thisShouldNotBeAStringButAMethod | | method classAndSelector | classAndSelector := ' ' split: thisShouldNotBeAStringButAMethod. method := (Smalltalk at: classAndSelector first asSymbol) methodNamed: classAndSelector second asSymbol. method recompile. ].
But if I remember it introduces a conflict with Fuel in MetalevelTests. I'm checking.
On 27 janv. 2013, at 17:19, Stéphane Ducasse wrote:
Camille I integrated your second version of the issue but I still get
testInstanceSideMethodsWithNilKeyInLastLiteral
self assert: SystemNavigation new instanceSideMethodsWithNilKeyInLastLiteral isEmpty
participants (3)
-
Marcus Denker -
Stéphane Ducasse -
wam