Bug #8676: ruby 2.0 can not require or load the source file with non-ascii path name

• Status: Closed
• Priority: Normal
• Assignee: -
• Target version: -
• ruby -v: ruby 2.0.0p277 (2013-07-23 revision 42121) [i386-mingw32]
• Backport: 1.9.3: UNKNOWN, 2.0.0: REQUIRED

[[e-mail] (贾 延平)]

=begin
Sorry for my poor english:)

I attached the patch to fix the problem, but I don't know is it the right way.

Changelog:
include/ruby/intern.h change the declaration of "rb_load_file"；change parameter from char to VALUE
*load.c change the caller
*ruby.c change the "rb_load_file"'s implement
*win32/file.c change the win32 api call from ANSI type to UNICODE type.
=end

[charliesome (Charlie Somerville)]

This patch looks good to me for trunk. It can't be backported though because that would break API compatibility.

[nobu (Nobuyoshi Nakada)]

• Subject changed from ruby 2.0 can not require or load the source file with utf-8 encoding and non-asii chars to ruby 2.0 can not require or load the source file with non-ascii path name
• Description updated (diff)
• Category changed from platform/mingw to platform/windows

[[e-mail] (贾 延平)]

By the way,the file nacl/pepper_main.c has a same name function at line 824.
Is the file need change too?

[[e-mail] (贾 延平)]

• File win32_file_open_patch.patch win32_file_open_patch.patch added

Update the patch
Using the encoding from path name

[nobu (Nobuyoshi Nakada)]

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

This issue was solved with changeset r42183.
贾, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

load.c: search in OS path encoding

• load.c (rb_load_internal): use rb_load_file_str() to keep path
  encoding.
• load.c (rb_require_safe): search in OS path encoding for Windows.
• ruby.c (rb_load_file_str): load file with keeping path encoding.
• win32/file.c (rb_file_load_ok): use WCHAR type API assuming incoming
  path is encoded in UTF-8. [ruby-core:56136] [Bug #8676]

[[e-mail] (贾 延平)]

There is a new bug from the change.
FILE's encoding is wrong.
In file ruby.c and function load_file_internal type VALUE convert to char*, and lost the encoding info
(({const char orig_fname = StringValueCStr(argp->fname);}))
and call the rb_parser_compile_file with the char variable
(({ tree = rb_parser_compile_file(parser, orig_fname, f, line_start);}))
and in file parse.y and function rb_parser_compile_file, type char* convert to VALUE using the filesystem encoding.
(({ return rb_parser_compile_file_path(vparser, rb_filesystem_str_new_cstr(f), file, start);}))
But the beginning parameter argp->fname's encoding not the filesystem encoding.

What's the principle of ruby's encoding?Why so many VALUE to char* converting?Is the char* has the regular encoding？

I think we should convert the encoding at the boundary and every thing should encoding to internal encoding in the internal, am I right?

[thomthom (Thomas Thomassen)]

This patch makes the file functions explicitly call the *W version of the file functions. Isn't it better to provide the UNICODE compile flag. http://msdn.microsoft.com/en-us/library/cc194801.aspx

CreateFile, and the likes, would then call CreateFileW instead of CreateFileA.

[usa (Usaku NAKAMURA)]

• Related to Bug #9699: Cannot require .so file on Windows if the file path is unicode (Includes patch) added

[usa (Usaku NAKAMURA)]

• Backport changed from 1.9.3: UNKNOWN, 2.0.0: UNKNOWN to 1.9.3: UNKNOWN, 2.0.0: REQUIRED