theseion / Fuel

Fuel, the Smalltalk object serializer
https://theseion.github.io/Fuel
MIT License
27 stars 13 forks source link

Installation Error on GToolkit #267

Open seandenigris opened 1 year ago

seandenigris commented 1 year ago

The underlying Pharo build is Pharo-10.0.0+build.521.sha.14f541319d443f4261f84f4fa19fbb34460a5edb (64 Bit)

Since I see Pharo 10 passing in the matrix, I wonder if this is something GT-specific...

First I get a weird error from Calypso about ZnBufferedReadStream>>fuelNextWordsInto: being moved to the FuelPlatformExtensions package. I'm not sure where that package even comes from since I don't see it in either the Pharo or Fuel repos.

Next, I get the following error:

Screenshot 2023-01-12 at 6 32 05 AM

At the top of the stack, you can see my implementation, which seems to have corrected the problem.

theseion commented 1 year ago

FuelPlatform is obviously broken in this instance. The implementation for Pharo 10 is

fileNamed: aFilename readStreamDo: aBlock
    ^ ((Smalltalk at: #File) named: aFilename) readStreamDo: [ :stream |
        stream binary.
        aBlock value: stream ]

How are you loading Fuel? Which version? Which version of Fuel is in the image (if any) before you load Fuel?

seandenigris commented 1 year ago

I’m loading 5.1.0 on top of whatever version comes with GT. I’ll update with the script and GT base version when I get to a computer

theseion commented 1 year ago

I just realised that the methods in your stack trace aren't actually from version 5 but from version 4. Clearly, version 5 wasn't properly loaded.

seandenigris commented 1 year ago

As I investigated, I noticed something similar. Trying to figure out why…

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will remain open but will probably not come into focus. If you still think this should receive some attention, leave a comment. Thank you for your contributions.