search

psvensson / golgotterath

An experimental isometric game engine written in Pharo Smalltalk.
MIT License
33 stars 3 forks source link

Very long method names causing load errors on Windows #3

Closed riverdusty closed 4 years ago

riverdusty commented 4 years ago

Failed to stat file 'C:/Users/river/Pharo/images/Pharo 8.0 - 64bit (stable)/pharo-local/iceberg/psvensson/PharoSound/SoundMorphicUserInterface.package/ScaleMorph.class/instance/start.stop.minorTick.minorTickLength.majorTick.majorTickLength.caption.tickPrintBlock.labelsAbove.captionAbove..st': The filename or extension is too long.

Should try rename method to something like 'startWithArgs:' and send in a collection of some sort

psvensson commented 4 years ago

Wow, right. I'm sorry to hear that, Dusty. This might be a bit problematic, since PharoSound is just a fork of the SmalltalkHub repo with the same name. The reason I made a fork was that I needed to add functionality to play sound that was not first read from file (I store sounds inside base64 encoded strings in methods instead). PahroSound is huge, however and I do not understand all of it, so it might be a quite big undertaking.

As you can see, I have not tested this yet on Windows :) But I will see if I can provide another release somehow which does not include sound, perhaps. I will give ti a think, any ideas welcome of course.

On Wed, Jan 29, 2020 at 12:51 PM Dusty notifications@github.com wrote:

Failed to stat file 'C:/Users/river/Pharo/images/Pharo 8.0 - 64bit (stable)/pharo-local/iceberg/psvensson/PharoSound/SoundMorphicUserInterface.package/ScaleMorph.class/instance/start.stop.minorTick.minorTickLength.majorTick.majorTickLength.caption.tickPrintBlock.labelsAbove.captionAbove..st': The filename or extension is too long.

Should try rename method to something like 'startWithArgs:' and send in a collection of some sort

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/psvensson/golgotterath/issues/3?email_source=notifications&email_token=AAAJVARM7IC64XMFADVCGR3RAFUSHA5CNFSM4KNDIUF2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IJQJBJQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAJVAULEMGQGJHEM36RU3LRAFUSHANCNFSM4KNDIUFQ .

-- Peter Svensson

http://se.linkedin.com/in/petersvensson (LinkedIn) http://twitter.com/psvensson (@psvensson)

psvensson commented 4 years ago

OK, this is not a very good solution, but I wonder if a temporary fix could be to create a new pharo image and give it a very short name, like 'Pharo8' instead of 'Pharo 8.0 - 64bit (stable)' and also move the Pharo folder up to the root of C:

This might be enough to be able to load the admittedly long classnames of PharoSound

psvensson commented 4 years ago

Hi again riverdusty! I've now read up a bit more and the way to solve this is to convert the source format on github to 'Tonel', so I just did that, which will make paths much shorter.

Please try again to install and let me know if you still have problems.

psvensson commented 4 years ago

Hi, I'm closing this issue now as it should have been resolved. Please reopen if you still have problems