Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add: Chinese Language #137

Open
wants to merge 6 commits into
base: default
Choose a base branch
from

Conversation

littlegolden
Copy link

Hi, the setup.iss file use Windows 1252 Character Encoding, the Hanzi can not display, can we change it into UTF-8 encoding?

@littlegolden
Copy link
Author

I know there are definitely some problems in this pr, please tell me and let me fix it.

@clechasseur
Copy link
Owner

Hi, the setup.iss file use Windows 1252 Character Encoding, the Hanzi can not display, can we change it into UTF-8 encoding?

I don't mind per se, as long as the ISS compiler accepts it.

@clechasseur
Copy link
Owner

I know there are definitely some problems in this pr, please tell me and let me fix it.

Let me know when it's ready for review. I'll convert it to Draft in the meantime.

@clechasseur clechasseur marked this pull request as draft February 13, 2021 20:01
@clechasseur
Copy link
Owner

Forgot to mention this earlier, but: thanks, that's awesome! 😄 I'll create an issue to track this in HISTORY.

@clechasseur clechasseur linked an issue Feb 14, 2021 that may be closed by this pull request
@littlegolden
Copy link
Author

Thank you, now you can review it~

@clechasseur clechasseur marked this pull request as ready for review February 15, 2021 00:39
@clechasseur clechasseur self-requested a review February 15, 2021 00:39
@@ -81,6 +81,7 @@ SignTool=certum
[Languages]
Name: en_CA; MessagesFile: compiler:Default.isl
Name: fr_CA; MessagesFile: compiler:Languages\French.isl
Name: zh-CN; MessagesFile: compiler:Languages\Chinese.isl
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the reason the build fails - the language name cannot have hyphens, so you must rename it to zh_CN (underscore instead of hyphen).

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll review the rest once I have more time. Thank you! 😄

Fix hyphens typo
@clechasseur
Copy link
Owner

It seems the build still fails on CI because it cannot find the "Chinese.isl" file from the Inno Setup's installation directory. Is this a regular file included with Inno or is it added by something else? I'm not on my PC so can't check if I have it locally or not...

@clechasseur
Copy link
Owner

By the way, sorry for my slow response speed, personal life is very hectic these days, but I still want to merge this.

@littlegolden
Copy link
Author

littlegolden commented Mar 2, 2021

It's okay, don't rush to merge 😃, we can wait for there is no problem and then merge it . Since I'm not familiar with the isl application 🤣, what I did is just found the places that might relates ti18n and modified them 🤣

@clechasseur
Copy link
Owner

It's okay, don't rush to merge 😃, we can wait for there is no problem and then merge it . Since I'm not familiar with the isl application 🤣, what I did is just found the places that might relates ti18n and modified them 🤣

I'll check when I'm on my PC to make sure this file exists. Perhaps it's named differently.

@clechasseur
Copy link
Owner

I've checked on my PC earlier and there's no official Chinese translation for Inno Setup. However, there is an unofficial one for Chinese (Simplified) here:

https://jrsoftware.org/files/istrans/

Try downloading this file and including it in your PR instead of referring to the one shipped with the compiler and see how it goes.

@clechasseur clechasseur force-pushed the default branch 2 times, most recently from a6f832e to 1e71272 Compare October 3, 2021 23:55
@littlegolden
Copy link
Author

ok, i'll update this pr ❤

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] Chinese language support
2 participants