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
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
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
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
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:
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
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
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,
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
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
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
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
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
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;
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
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
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
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
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
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:
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
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
22 matches
Mail list logo