Bug #10493: Wrong encoding for Etc.getlogin in Windows

• Status: Closed
• Assignee: -
• Target version: 2.2.0
• ruby -v: ruby 2.1.4p265 (2014-10-27 revision 48166) [x64-mingw32]
• Backport: 2.0.0: DONE, 2.1: DONE

[davispuh (Dāvis Mosāns)]

name = Etc.getlogin # I've Windows username "Dāvis"
puts name           # D�vis
puts name.inspect   # "D\xE2vis"
puts name.encoding  # ASCII-8BIT
name.force_encoding('Windows-1257')
puts name           # Dāvis

As you can see \xE2 isn't "ā" in ASCII, but it is for Windws-1257 encoding.

[nobu (Nobuyoshi Nakada)]

• Description updated (diff)
• Category set to ext
• Status changed from Open to Feedback
• Target version set to 2.2.0
• Backport changed from 2.0.0: UNKNOWN, 2.1: UNKNOWN to 2.0.0: REQUIRED, 2.1: REQUIRED

Is your Encoding.default_external Windows-1257?

[nobu (Nobuyoshi Nakada)]

• Status changed from Feedback to Closed
• % Done changed from 0 to 100

Applied in changeset r48360.

---

win32.c: login name encoding

• ext/etc/etc.c (etc_getlogin): set login name encoding properly.
  [ruby-core:66163] [Bug #10493]

[davispuh (Dāvis Mosāns)]

No, Encoding.default_external is based on current consoles encoding. And by default it is IBM775, but can be different for each opened console. Also it's not related to Encoding which is used by Windows ANSI API. Windows-1257 encoding is set in Control Panel for non-Unicode programs and will be used for all Windows ANSI functions. Proper way is to always use Windows Unicode/Wide API and then it will be encoded with UTF-16.

Thanks for patch, I'll test it when it will be released.

[duerst (Martin Dürst)]

Dāvis Mosāns wrote:

No, Encoding.default_external is based on current consoles encoding. And by default it is IBM775, but can be different for each opened console. Also it's not related to Encoding which is used by Windows ANSI API. Windows-1257 encoding is set in Control Panel for non-Unicode programs and will be used for all Windows ANSI functions.

Can you say which section of the control panel that is?

Proper way is to always use Windows Unicode/Wide API and then it will be encoded with UTF-16.

Thanks for patch, I'll test it when it will be released.

It would be highly desirable to test it before it's released.

[davispuh (Dāvis Mosāns)]

Control Panel -> Region -> Administrative -> Change system locale

Here's article describing how to change locale for Windows 7 and 8

For Windows 7

For Windows 8

To test it now I would need to setup environment and compile Ruby, but I never have done that and currently don't really have time for it. Anyway I'll see maybe I will be able to find time for it someday.

[luislavena (Luis Lavena)]

Davis,

You can build Ruby easily by using RubyInstaller recipes:

https://github.com/oneclick/rubyinstaller

You will also need a clone of Ruby source code and usage of LOCAL to indicate the PATH where to find Ruby source code.

See examples in RubyInstaller's readme.

Cheers.

[davispuh (Dāvis Mosāns)]

Thanks, that was surprisingly easy. Only trunk didn't build so I compiled from ruby_2_1 branch and cherry-picked r48360, r48364
Now I can confirm that patch works.

name = Etc.getlogin
puts name.inspect   # "D\u0101vis"
puts name.encoding  # UTF-8

But looks like encoding for string outputting have been broken somewhere (I hope it's fixed in some commits not yet merged in this branch)

puts "\u0101"
=> ā   # on released Ruby 2.1.4p265
=> Ä?  # on 2.1.5p272, I just compiled from ruby_2_1 branch

It used to be that when outputting string it's encoded to Encoding.default_external, but that's not the case for this.

[usa (Usaku NAKAMURA)]

• Backport changed from 2.0.0: REQUIRED, 2.1: REQUIRED to 2.0.0: DONE, 2.1: REQUIRED

Backported into ruby_2_0_0 at r48634.

memo:
I changed the patch a little, because 2.0.0 doesn't have rb_w32_wstr_to_mbstr().
2.1 has the function, so I guess it's no problem to backport to 2.1 from trunk.

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 2.0.0: DONE, 2.1: REQUIRED to 2.0.0: DONE, 2.1: DONE

Backported into ruby_2_1 at r49473.

usa-san, thank you for your advice. I'll keep my eyes on RubyCI, especially on vc-x64.

[usa (Usaku NAKAMURA)]

Tomoyuki Chikanaga wrote:

Backported into ruby_2_1 at r49473.
usa-san, thank you for your advice. I'll keep my eyes on RubyCI, especially on vc-x64.

It seems good. (I've tested on my box.)
Thank you!