Post by sidrek in TPE Version 10.1 Released For Supporters comments

TPE Version 10.1 Released For Supporters » Comments

Viewing post in TPE Version 10.1 Released For Supporters comments

Hi Proxxie,

Fantastic games, thanks.

I would like to report a little bug, in the 10.1. (Linux Operating System), I obtained during the game the below error (see the screen).

This happen because the Linux file system is case sensitive, and the game try to load the image g_Bunnies_Jovi_Portrait1.png (please note the first char lowercase) instead of the G_Bunnies_Jovi_Portrait1.png (please note the first char uppercase).

I fixed crating a copy of the asset changing the name.

Thanks again, regards.
S.

Proxxie1 year ago (1 edit)

This has been a maddening one to deal with. I'm not a Linux person, but let me ask - is the default Linux filesystem case-sensitive? I'm not a Linux person so I really don't know.

I've searched all my references and haven't found one to lowercase "g". I think it's because I fixed it a long time ago in my version, but the tool you use to upload to Itch just looks for diffs, and maybe it's not registering that as a diff because it isn't case-sensitive?

I'll take another look T_T

Thanks for the report and the kind words, btw!

sidrek1 year ago (2 edits) (+1)

Hi Proxxie,

> but let me ask - is the default Linux filesystem case-sensitive?

Yes, in the same folder you can have two files, for example test.TXT and test.txt.
I know, this should be a little confusing.

> but the tool you use to upload to Itch just looks for diffs, and maybe it's not registering that as a diff because it isn't case-sensitive?

Wait I lost, you talk about which tools?
I reported the error because I encounter the "Error: Failed to load: img ..." as for the screen in my previous message.

UPDATE: I checked more in depth and the "misconfiguration" seems to be in the Actor.json file, where there there is a reference to the "g_Bunnies_Jovi_Portrait1", changing it not seems enough to fix the problem.
Maybe the file must be "compiled"?
Anyway, if in your recent code you don't have any similar reference, almost for sure you already fixed the problem.

I hope this can help you :-)

Thanks and regards.
S.

Proxxie1 year ago

Wow, good catch! Thank you very much! I had not fixed it, as it turns out. It's fixed now! This is a very old bug - I've never been able to replicate it, so it was hard to track down.

The engine doesn't do any real compilation (just JavaScript JIT compilation) - I'm guessing there's probably a cache or something that makes the error persist on your machine after you fixed it.

The fix probably won't be in the public version until 10.2 or 10.3, but it's coming. Thanks again!

sidrek1 year ago(+1)

I'm glad to helped you :-)

sidrek1 year ago (2 edits) (+1)

Hi Proxxie,

Sorry again for bothering you, I founded another two cases of the same error, below the screenshots.

In this case the problem is with the image Ogremarsh_People_7, but in different json (Map199, Map201, Map211, and so on ) referred as ogremarsh_people_7 (all in lowercase), the same problem also for the image Ogremarsh_People_Dmg_2, referred in the json (Map222), like ogremarsh_people_dmg_2 (also in this case all in lowercase).

Please note in the folder img/characthers, there are the assets ogremarsh_people_8.png and ogremarsh_people_10.png that are perfectly mapped with the json, but maybe should be rename for naming convention with the others files.

Regards,
S.

Proxxie1 year ago

Thanks for the report! I'll get 'em fixed.

As an aside, macOS' default is non-case-sensitive, but it gives you the option of choosing a case-sensitive filesystem. All the advice I read on the subject said "Don't choose case-sensitive unless you have a specific need for it." I'm now beginning to understand why lol.

Thank you for scrubbing through the JSON to find 'em! That makes them really easy to fix :D

Proxxie1 year ago

Actually, I may have to automate that. There's probably too many references to those files to do by hand. But I have a tool that should be able to do it if I play around with it a bit (the thing I wrote when I was looking to convert the project to MZ.)

itch.io

Viewing post in TPE Version 10.1 Released For Supporters comments