:) iep, I would like to say 10 builds (but this is 9).

Jannik

On May 4, 2011, at 15:08 , Andre Hora wrote:

It is related to the integration with Glamour. I can have a look.
ps: not 10 days, 5 days ;)

On Wed, May 4, 2011 at 2:59 PM, jannik.laval <jannik.laval@gmail.com> wrote:
Good and bad news :)

That is a good news for my changes.
The bad news is that the bug is 10 days old...
Anyone know eyesee better than me to fix it ?

Cheers,
Jannik

On May 4, 2011, at 14:53 , Andre Hora wrote:

Jannik,  I see in Hudson that we have this error with EyeSee since #330 Apr 29, 2011 3:57:59 PM
So, I think it is not related with your change.

On Wed, May 4, 2011 at 2:34 PM, jannik.laval <jannik.laval@gmail.com> wrote:
I am not sure that eyesee is related.
You can take an image and see the report on hudson:

Jannik

On May 4, 2011, at 12:28 , Andre Hora wrote:

Hello,

There are no tests in EyeSee related to import model from MSE stream.
Jannik could you send me an image that I can reproduce this error in EyeSee? (I should only run EyeSee tests in this images?

regards,

On Wed, May 4, 2011 at 11:56 AM, jannik.laval <jannik.laval@gmail.com> wrote:
for eyesee, I do not know.
For smallDude, a test import a model from a MSE Stream.

I think that this case should be tested in MSE importer tests.

Jannik


On May 4, 2011, at 11:20 , Simon Denier wrote:

>
> Looks bad in my opinion. Why is there some dependency between MSE import and smalldude/eyesee tests?
>
>
> On 4 mai 2011, at 09:14, Laval Jannik wrote:
>
>> Hi,
>>
>> I changed the behavior of FAMIXMethod>>parentPackage.
>>
>> In fact, when a method is in the same package than its class, its parentPackage should be nil.
>>
>> So, to not break the behavior, I changed the accessor like this:
>>
>> ===
>> parentPackage
>>      ^ parentPackage ifNil:[ self belongsTo parentPackage]
>> ===
>>
>> But this breaks during importation of MSE files (it breaks some tests of smalldude and eyesee).
>> The problem is due to the class that is not already defined during the importation.
>> So I changed the source code like that. I am not convince of the efficiency of this code.
>> Have you another idea ?
>>
>> ===
>> parentPackage
>>
>>      ^ parentPackage ifNil:[
>>              |cl|
>>              cl := self belongsTo.
>>              cl isNil
>>                      ifTrue:[nil]
>>                      ifFalse:[cl parentPackage]
>>      ]
>> ===
>>
>> ---
>> Jannik Laval
>>
>>
>> _______________________________________________
>> Moose-dev mailing list
>> Moose-dev@iam.unibe.ch
>> https://www.iam.unibe.ch/mailman/listinfo/moose-dev
>
> --
> Simon Denier
>
>
>
>
> _______________________________________________
> Moose-dev mailing list
> Moose-dev@iam.unibe.ch
> https://www.iam.unibe.ch/mailman/listinfo/moose-dev


_______________________________________________
Moose-dev mailing list
Moose-dev@iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev



--
Andre Hora

_______________________________________________
Moose-dev mailing list
Moose-dev@iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev


_______________________________________________
Moose-dev mailing list
Moose-dev@iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev




--
Andre Hora

_______________________________________________
Moose-dev mailing list
Moose-dev@iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev


_______________________________________________
Moose-dev mailing list
Moose-dev@iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev




--
Andre Hora

_______________________________________________
Moose-dev mailing list
Moose-dev@iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev