Smalltalk › Pharo › Pharo Smalltalk Developers

Fuel Not Serializing Stateful Traits

_‹ Previous Topic Next Topic _›

Classic

List

Threaded

7 messages Options

Sean P. DeNigris

Fuel Not Serializing Stateful Traits

Administrator

Pharo 7.0.3. Fuel 3.0.0 loaded per readme via:
Metacello new
repository: 'github://theseion/Fuel:3.0.0/repository';
baseline: 'Fuel';
load. "Also tried loading : #('CoreWithExtras')"

I see commit 1dd... back in March 2018 says it should work, but I'm
consistently getting the following error:
Array(Object)>>errorSubscriptBounds:
Array(Object)>>at:
SpdClass class(ClassDescription)>>instanceVariableNamesDo:
FLVariablesMapping>>instanceVariableNamesToSerialize
FLVariablesMapping>>initializeAnalyzing
FLVariablesMapping class>>newAnalyzing:references:

-----
Cheers,
Sean
--
Sent from: http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.html

Cheers,
Sean

Sean P. DeNigris

Re: Fuel Not Serializing Stateful Traits

Administrator

Sean P. DeNigris wrote
> I see commit 1dd... back in March 2018 says it should work

Investigating a bit further, maybe the problem is in Pharo, because another
test in Pharo 8 didn't show the issue. instanceVariableNamesDo: doesn't seem
to have changed, so I guess it's deeper. Any ideas? Otherwise I'll have to
rollback my stateful trait usage until Pharo 8!

-----
Cheers,
Sean
--
Sent from: http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.html

Cheers,
Sean

alistairgrant

Re: Fuel Not Serializing Stateful Traits

Hi Sean,

On Mon, 23 Sep 2019 at 23:37, Sean P. DeNigris <[hidden email]> wrote:
>
> Sean P. DeNigris wrote
> > I see commit 1dd... back in March 2018 says it should work
>
> Investigating a bit further, maybe the problem is in Pharo, because another
> test in Pharo 8 didn't show the issue. instanceVariableNamesDo: doesn't seem
> to have changed, so I guess it's deeper. Any ideas? Otherwise I'll have to
> rollback my stateful trait usage until Pharo 8!

This might provide a bit more info:
https://github.com/pharo-project/pharo/issues/3546

Cheers,
Alistair

Max Leske

Re: Fuel Not Serializing Stateful Traits

Hi,

As I wrote to Sean on Discord, Marcus fixed that bug during ESUG. I
realize now, however, that it is only fixed in Pharo 8. I would need to
write a patch for Fuel to make it work in Pharo 7.

Alternatively, you could include the patch in your code, then Fuel
should work.

Cheers,
Max

On 24 Sep 2019, at 7:59, Alistair Grant wrote:

> Hi Sean,
>
>
> On Mon, 23 Sep 2019 at 23:37, Sean P. DeNigris <[hidden email]>
> wrote:
>>
>> Sean P. DeNigris wrote
>>> I see commit 1dd... back in March 2018 says it should work
>>
>> Investigating a bit further, maybe the problem is in Pharo, because
>> another
>> test in Pharo 8 didn't show the issue. instanceVariableNamesDo:
>> doesn't seem
>> to have changed, so I guess it's deeper. Any ideas? Otherwise I'll
>> have to
>> rollback my stateful trait usage until Pharo 8!
>
> This might provide a bit more info:
> https://github.com/pharo-project/pharo/issues/3546
>
> Cheers,
> Alistair

Marcus Denker-4

Re: Fuel Not Serializing Stateful Traits

we coud back port the fix...

> On 24 Sep 2019, at 08:38, Max Leske <[hidden email]> wrote:
>
> Hi,
>
> As I wrote to Sean on Discord, Marcus fixed that bug during ESUG. I realize now, however, that it is only fixed in Pharo 8. I would need to write a patch for Fuel to make it work in Pharo 7.
>
> Alternatively, you could include the patch in your code, then Fuel should work.
>
>
> Cheers,
> Max
>
>
> On 24 Sep 2019, at 7:59, Alistair Grant wrote:
>
>> Hi Sean,
>>
>>
>> On Mon, 23 Sep 2019 at 23:37, Sean P. DeNigris <[hidden email]> wrote:
>>>
>>> Sean P. DeNigris wrote
>>>> I see commit 1dd... back in March 2018 says it should work
>>>
>>> Investigating a bit further, maybe the problem is in Pharo, because another
>>> test in Pharo 8 didn't show the issue. instanceVariableNamesDo: doesn't seem
>>> to have changed, so I guess it's deeper. Any ideas? Otherwise I'll have to
>>> rollback my stateful trait usage until Pharo 8!
>>
>> This might provide a bit more info:
>> https://github.com/pharo-project/pharo/issues/3546
>>
>> Cheers,
>> Alistair
>

Sean P. DeNigris

Re: Fuel Not Serializing Stateful Traits

Administrator

Marcus Denker-4 wrote
> we coud back port the fix...

That would be great if you think it's fairly safe. Although personally I
could roll back my stateful traits use until P8 since I just started using
them. BTW what is the ETA for P8 release?

-----
Cheers,
Sean
--
Sent from: http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.html

Cheers,
Sean

ducasse

Re: Fuel Not Serializing Stateful Traits

> On 24 Sep 2019, at 14:54, Sean P. DeNigris <[hidden email]> wrote:
>
> Marcus Denker-4 wrote
>> we coud back port the fix...
>
> That would be great if you think it's fairly safe. Although personally I
> could roll back my stateful traits use until P8 since I just started using
> them. BTW what is the ETA for P8 release?

beta end of nov.
release end of feb?

Unclear yet
>
>
>
> -----
> Cheers,
> Sean
> --
> Sent from: http://forum.world.st/Pharo-Smalltalk-Developers-f1294837.html
>