aarcangeli / Serious-Sam-Android

Porting of Serious Sam: The Second Encounter for android
72 stars 23 forks source link

Cyrillic symbols not support in paths and filenames #32

Open Skyrimus opened 4 years ago

Skyrimus commented 4 years ago

Screenshot_20191006-195654 SeriousSam.log

Skyrimus commented 4 years ago

Trouble with symbols encoding, I tried change it to utf-8, but it not works

aarcangeli commented 4 years ago

The string "Текстуры" where does it come from? I suggest not using non-Latin file names, is it required for something?

Skyrimus commented 4 years ago

This string come from Russian modification, in this modification used Russian language to Textures, on Windows version this mod works fine, I think it will need fix

Skyrimus commented 4 years ago

In path to some textures used cyrillic symbols as Russian language

Skyrimus commented 4 years ago

Do you have some ideas to fix it? If I change homeDir path SeriousSam https://github.com/Skyrimus/Serious-Sam-Android/blob/master/app/src/main/java/com/github/aarcangeli/serioussamandroid/MainActivity.java#L498 to some cyrillic symbols it works fine

aarcangeli commented 4 years ago

Where is stored Tp.tex? in a gro file?

Skyrimus commented 4 years ago

Change SeriousSam path to Сэм path works, but path to Текстуры not works Screenshot_20191009-004157 Screenshot_20191009-004342

Skyrimus commented 4 years ago

@aarcangeli yeah, in gro, and i tried to unzip Текстуры to game path and it not works too

Skyrimus commented 4 years ago

This some more ?????? is Текстуры have not utf encoding i think and it not works, but Сэм is utf

Skyrimus commented 4 years ago

How to change path to Текстуры in utf encoding?

Skyrimus commented 4 years ago

@aarcangeli How can I contact you? Facebook?

aarcangeli commented 4 years ago

How about telegram? my username is http://t.me/alessandroarcangeli

Skyrimus commented 4 years ago

@aarcangeli ok.

Skyrimus commented 4 years ago

@aarcangeli i can t write you, please add me https://t.me/Skyrimus

aarcangeli commented 4 years ago

This is now fixed on master branch