Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-29 Thread Tudor Girba
Thanks for integrating the fix. The Moose builds are back to normal. Cheers, Doru On Tue, Jan 28, 2014 at 4:41 PM, Martin Dias tinchod...@gmail.com wrote: Done. https://pharo.fogbugz.com/f/cases/12731/Traits-modifications-cause-a-DNU On Tue, Jan 28, 2014 at 2:13 PM, Tudor Girba

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Nicolai Hess
2014-01-27 Martin Dias tinchod...@gmail.com On Mon, Jan 27, 2014 at 7:10 AM, Tudor Girba tu...@tudorgirba.com wrote: I opened an issue: https://pharo.fogbugz.com/f/cases/12731/Traits-modifications-cause-a-DNU I marked it as a show stopper because right now you cannot load a package with

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Martin Dias
So, what to do? - Don't send ClassModifiedClassDefinition in SystemAnnouncertraitDefinitionChangedFrom: oldTrait to: newTrait ? - Implement Traitlayout ? - Test for oldClassDefinition isTrait in ClassModifiedClassDefinitionisPropagating ? I discussed with Camille and we think it's

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Esteban Lorenzano
yes, please :) On 28 Jan 2014, at 13:47, Martin Dias tinchod...@gmail.com wrote: So, what to do? - Don't send ClassModifiedClassDefinition in SystemAnnouncertraitDefinitionChangedFrom: oldTrait to: newTrait ? - Implement Traitlayout ? - Test for oldClassDefinition isTrait in

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Tudor Girba
Yes, please! Doru On Tue, Jan 28, 2014 at 2:04 PM, Esteban Lorenzano esteba...@gmail.comwrote: yes, please :) On 28 Jan 2014, at 13:47, Martin Dias tinchod...@gmail.com wrote: So, what to do? - Don't send ClassModifiedClassDefinition in SystemAnnouncertraitDefinitionChangedFrom:

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Martin Dias
hhahah, ok, I'm on it On Tue, Jan 28, 2014 at 2:06 PM, Tudor Girba tu...@tudorgirba.com wrote: Yes, please! Doru On Tue, Jan 28, 2014 at 2:04 PM, Esteban Lorenzano esteba...@gmail.comwrote: yes, please :) On 28 Jan 2014, at 13:47, Martin Dias tinchod...@gmail.com wrote: So, what to

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Tudor Girba
Thank you! I can review it tomorrow morning. Doru On Tue, Jan 28, 2014 at 2:10 PM, Martin Dias tinchod...@gmail.com wrote: hhahah, ok, I'm on it On Tue, Jan 28, 2014 at 2:06 PM, Tudor Girba tu...@tudorgirba.com wrote: Yes, please! Doru On Tue, Jan 28, 2014 at 2:04 PM, Esteban

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Martin Dias
Done. https://pharo.fogbugz.com/f/cases/12731/Traits-modifications-cause-a-DNU On Tue, Jan 28, 2014 at 2:13 PM, Tudor Girba tu...@tudorgirba.com wrote: Thank you! I can review it tomorrow morning. Doru On Tue, Jan 28, 2014 at 2:10 PM, Martin Dias tinchod...@gmail.com wrote: hhahah, ok,

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Benjamin
It’s a duplicate of a closed issue (recently integrated by Sven) We should maybe join forces :) Ben On 28 Jan 2014, at 12:41, Martin Dias tinchod...@gmail.com wrote: Done. https://pharo.fogbugz.com/f/cases/12731/Traits-modifications-cause-a-DNU On Tue, Jan 28, 2014 at 2:13 PM, Tudor

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Tudor Girba
Could you be more specific? Which issue? Doru On Tue, Jan 28, 2014 at 4:50 PM, Benjamin benjamin.vanryseghem.ph...@gmail.com wrote: It's a duplicate of a closed issue (recently integrated by Sven) We should maybe join forces :) Ben On 28 Jan 2014, at 12:41, Martin Dias

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Tudor Girba
Thanks, Martin. I will look tomorrow morning. Doru On Tue, Jan 28, 2014 at 4:41 PM, Martin Dias tinchod...@gmail.com wrote: Done. https://pharo.fogbugz.com/f/cases/12731/Traits-modifications-cause-a-DNU On Tue, Jan 28, 2014 at 2:13 PM, Tudor Girba tu...@tudorgirba.com wrote: Thank

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-28 Thread Pharo4Stef
may be in addition we could add layout returning a default value. Stef On 28 Jan 2014, at 13:47, Martin Dias tinchod...@gmail.com wrote: So, what to do? - Don't send ClassModifiedClassDefinition in SystemAnnouncertraitDefinitionChangedFrom: oldTrait to: newTrait ? - Implement Traitlayout

[Pharo-dev] strange error in latest 3.0 -

2014-01-26 Thread Tudor Girba
Hi, If I execute the following code in 30720: Gofer new smalltalkhubUser: 'Moose' project: 'MooseAlgos'; configuration; loadDevelopment. I get an emergency evaluator with a strange Original error: MessageNotUnderstood Traitlayout. Can anyone have a look at the first one? It seems to be related

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-26 Thread Esteban Lorenzano
yep, I can confirm. Looks like latest update broke something. Esteban On 26 Jan 2014, at 15:50, Tudor Girba tu...@tudorgirba.com wrote: Hi, If I execute the following code in 30720: Gofer new smalltalkhubUser: 'Moose' project: 'MooseAlgos'; configuration;

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-26 Thread Tudor Girba
I looked more into it, and here is a smaller example: Gofer new smalltalkhubUser: 'Moose' project: 'MooseAlgos'; package: 'Moose-Algos-Graph'; load This one does open the debugger, so the other emergency evaluator issue comes from somewhere else. The problem seems to come from a Trait being

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-26 Thread Tudor Girba
I looked some more. If I remove the check from the below message it seems to work fine (essentially, re-establishing the code from before). MCPackageManager classclassModified: anEvent anEvent isPropagation ifTrue: [ ^ self ]. self managersForClass: anEvent classAffected do:[ :mgr | mgr

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-26 Thread Nicolai Hess
2014-01-26 Tudor Girba tu...@tudorgirba.com Hi, If I execute the following code in 30720: Gofer new smalltalkhubUser: 'Moose' project: 'MooseAlgos'; configuration; loadDevelopment. I get an emergency evaluator with a strange Original error: MessageNotUnderstood Traitlayout. Can

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-26 Thread Tudor Girba
Indeed, I figured that one out, although after I load the code, there is nothing like this (the method is removed from the package). Anyway, the real problem is the Trait one, like I described in my follow up emails. Cheers, Doru On Sun, Jan 26, 2014 at 5:00 PM, Nicolai Hess nicolaih...@web.de

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-26 Thread Martin Dias
On Sun, Jan 26, 2014 at 4:07 PM, Tudor Girba tu...@tudorgirba.com wrote: I looked more into it, and here is a smaller example: Gofer new smalltalkhubUser: 'Moose' project: 'MooseAlgos'; package: 'Moose-Algos-Graph'; load This one does open the debugger, so the other emergency evaluator

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-26 Thread Martin Dias
On Sun, Jan 26, 2014 at 10:50 PM, Martin Dias tinchod...@gmail.com wrote: On Sun, Jan 26, 2014 at 4:07 PM, Tudor Girba tu...@tudorgirba.com wrote: I looked more into it, and here is a smaller example: Gofer new smalltalkhubUser: 'Moose' project: 'MooseAlgos'; package:

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-26 Thread Esteban Lorenzano
can you propose a fix? On 26 Jan 2014, at 22:56, Martin Dias tinchod...@gmail.com wrote: On Sun, Jan 26, 2014 at 10:50 PM, Martin Dias tinchod...@gmail.com wrote: On Sun, Jan 26, 2014 at 4:07 PM, Tudor Girba tu...@tudorgirba.com wrote: I looked more into it, and here is a smaller

Re: [Pharo-dev] strange error in latest 3.0 -

2014-01-26 Thread Tudor Girba
I opened an issue: https://pharo.fogbugz.com/f/cases/12731/Traits-modifications-cause-a-DNU I marked it as a show stopper because right now you cannot load a package with Traits in the image. I am not sure about triggering the event for subclasses, but I can certainly understand that it can be