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

MakeNSIS does not output unicode #19

Open
GoogleCodeExporter opened this issue Mar 14, 2015 · 3 comments
Open

MakeNSIS does not output unicode #19

GoogleCodeExporter opened this issue Mar 14, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

It seems like makensis does a conversion to char* at some point, a simple 
UTF16LE .nsi with some non-ascii characters do not make it all the way from 
input to output without a conversion that corrupts it.

Test script:
!echo "ĦĔĹĽΘ ŴØŖĿĎ" ;I don't know if the issue tracker can handle 
this so I also attached the script
!error AbortTest

Output from makensis.exe test.nsi:
--snipped--
Processing script file: "test.nsi"
????? ?╪??? (test.nsi:1)
!error: AbortTest
--snipped--

type test.nsi shows the correct output.

cmd.exe is running with a true type font (Lucida Console). I also tried running 
chcp 65001 first but that did not help...


Original issue reported on code.google.com by ander...@users.sourceforge.net on 11 Mar 2012 at 8:21

Attachments:

@GoogleCodeExporter
Copy link
Author

The problem is not with makensis but with cmd.exe. The cmd.exe can't really 
handle Unicode. You have to use makensisw.exe to see the Unicode output.

Original comment by j...@scratchpaper.com on 12 Mar 2012 at 3:21

  • Changed state: Invalid

@GoogleCodeExporter
Copy link
Author

Saying the console cannot handle Unicode is simply not true, it cannot handle 
RTL and things that require Uniscribe but it can display a lot of the BMP.

See http://i39.tinypic.com/2guxjrp.png

See also: Anyone who says the console can't do Unicode isn't as smart as they 
think they are: http://blogs.msdn.com/b/michkap/archive/2010/04/07/9989346.aspx

Original comment by ander...@users.sourceforge.net on 13 Mar 2012 at 12:42

@GoogleCodeExporter
Copy link
Author

It looks like you are right it looks like WriteConsoleW is your friend. I will 
have to take a look to see how to best fit it into the code. I'll keep this 
issue open and consider it for the next release.

Original comment by j...@scratchpaper.com on 13 Mar 2012 at 1:09

  • Changed state: Accepted

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

No branches or pull requests

1 participant