Project

General

Profile

Bug #13863

RUBYLIB にカレントディレクトリを設定した場合、-E cp932:utf-8 を指定するとスクリプトが全く実行されない

Added by dogatana (Toshihiko Ichida) 11 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
-
ruby -v:
ruby 2.4.1p111 (2017-03-22 revision 58053) [i386-mingw32]
[ruby-dev:50221]

Description

こんにちは。

Windows 環境の ruby での話です。

http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-list/49186
https://bugs.ruby-lang.org/issues/7881

について、これは少なくとも

  • ruby 2.3.3p222 (2016-11-21 revision 56859) [i386-mingw32]
  • ruby 2.4.1p111 (2017-03-22 revision 58053) [i386-mingw32]

では発生せず、対策されているようです。

ただし、

  • 環境変数 RUBYLIBにカンレントディレクトリ(.)を設定
  • -E cp932:utf-8 を指定

して上のリンクにある a.rb を実行すると、

ruby 2.3.3p222 (2016-11-21 revision 56859) [i386-mingw32] では

> ruby -E cp932:utf-8 a.rb
<internal:enc/prelude>:4:in `require': stack level too deep (SystemStackError)
        from <internal:enc/prelude>:4:in `<internal:enc/prelude>'

と表示され、即座に終了し、
ruby 2.4.1p111 (2017-03-22 revision 58053) [i386-mingw32] では

> ruby -E cp932:utf-8 a.rb

と何も表示せず、ruby が終了します。
(内部では SystemStackError が発生?)

-E オプションは標準出力を cp932 で出力したいためで、今はIO#set_encoding を使用するようにして対処していますが、
本質的には ruby 側での対策が望ましいのではと思います。

いかがでしょうか。

市田

Associated revisions

Revision 7a693278
Added by nobu (Nobuyoshi Nakada) 8 months ago

load.c: cwd encoding

  • load.c (rb_get_expanded_load_path): save cwd cache in OS path encoding, to get rid of unnecessary conversion and infinite loading when it needs encoding conversion. [Bug #13863]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60743 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 60743
Added by nobu (Nobuyoshi Nakada) 8 months ago

load.c: cwd encoding

  • load.c (rb_get_expanded_load_path): save cwd cache in OS path encoding, to get rid of unnecessary conversion and infinite loading when it needs encoding conversion. [Bug #13863]

Revision 60743
Added by nobu (Nobuyoshi Nakada) 8 months ago

load.c: cwd encoding

  • load.c (rb_get_expanded_load_path): save cwd cache in OS path encoding, to get rid of unnecessary conversion and infinite loading when it needs encoding conversion. [Bug #13863]

Revision 2368dbdf
Added by nagachika (Tomoyuki Chikanaga) 4 months ago

merge revision(s) 58745,58780,59040,60743: [Backport #13863]

rb_w32_ugetcwd: UTF-8 version getcwd

* dir.c (rb_dir_getwd): convert from UTF-8.

* win32/win32.c (w32_getcwd): codepage aware getcwd using
  GetCurrentDirectoryW.
potential memory leak

* dir.c (rb_dir_getwd): get rid of potential memory leak.

* util.c (ruby_getcwd): ditto.
file.c: realpath in OS path encoding

* dir.c (rb_dir_getwd_ospath): return cwd path in the OS path
  encoding.

* file.c (rb_realpath_internal): work in the OS path encoding
load.c: cwd encoding

* load.c (rb_get_expanded_load_path): save cwd cache in OS path
  encoding, to get rid of unnecessary conversion and infinite
  loading when it needs encoding conversion.
   [Bug #13863]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_4@62784 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 62784
Added by nagachika (Tomoyuki Chikanaga) 4 months ago

merge revision(s) 58745,58780,59040,60743: [Backport #13863]

rb_w32_ugetcwd: UTF-8 version getcwd

* dir.c (rb_dir_getwd): convert from UTF-8.

* win32/win32.c (w32_getcwd): codepage aware getcwd using
  GetCurrentDirectoryW.
potential memory leak

* dir.c (rb_dir_getwd): get rid of potential memory leak.

* util.c (ruby_getcwd): ditto.
file.c: realpath in OS path encoding

* dir.c (rb_dir_getwd_ospath): return cwd path in the OS path
  encoding.

* file.c (rb_realpath_internal): work in the OS path encoding
load.c: cwd encoding

* load.c (rb_get_expanded_load_path): save cwd cache in OS path
  encoding, to get rid of unnecessary conversion and infinite
  loading when it needs encoding conversion.
   [Bug #13863]

Revision 2171d75e
Added by nagachika (Tomoyuki Chikanaga) 4 months ago

merge revision(s) 57484,58767,58938,59041: [Backport #13863]

ruby.c: forbid options

* ruby.c (forbid_setid): constified.

* ruby.c (process_options): forbid if setid earlier.

ruby.c: encode script name

* ruby.c (process_options): encode script name to locale encoding
  instead of associate, if UTF-8 path.

ruby.c: file in load_file argument

* ruby.c (load_file): move opened file to an argument, to reduce
  open/close calls in the near future.

ruby.c: script name in UTF-8

* ruby.c (process_options): keep script name in UTF-8 if UTF8_PATH
  to get rid of loss by conversion.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_4@62797 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 62797
Added by nagachika (Tomoyuki Chikanaga) 4 months ago

merge revision(s) 57484,58767,58938,59041: [Backport #13863]

ruby.c: forbid options

* ruby.c (forbid_setid): constified.

* ruby.c (process_options): forbid if setid earlier.

ruby.c: encode script name

* ruby.c (process_options): encode script name to locale encoding
  instead of associate, if UTF-8 path.

ruby.c: file in load_file argument

* ruby.c (load_file): move opened file to an argument, to reduce
  open/close calls in the near future.

ruby.c: script name in UTF-8

* ruby.c (process_options): keep script name in UTF-8 if UTF8_PATH
  to get rid of loss by conversion.

Revision 3c5452db
Added by nagachika (Tomoyuki Chikanaga) 4 months ago

revert r62797, r62784. [Bug #13863]

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_4@62801 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 62801
Added by nagachika (Tomoyuki Chikanaga) 4 months ago

revert r62797, r62784. [Bug #13863]

History

#1 [ruby-dev:50222] Updated by usa (Usaku NAKAMURA) 11 months ago

私の手元にあるmswin32/mswin64/mingw32/mingw64の2.3.4/2.4.1/trunkではいずれでも再現しませんでした。
他に何か再現条件があるのでしょうか?

#2 [ruby-dev:50224] Updated by dogatana (Toshihiko Ichida) 11 months ago

仕事で使用している Windows 7 64bit の環境で試しましたが、現象発生します。
こちらで確認している条件は次の2点です。

  • カレントディレクトに日本語を含む
  • RUBYLIB に . 設定されている

ログを貼ります(プロンプトが2行になっています)が、どうでしょうか。

C:\usr\sandbox\ruby\req\テスト
$type a.rb
puts 'a'
require'./b'

C:\usr\sandbox\ruby\req\テスト
$type b.rb
puts 'b'

C:\usr\sandbox\ruby\req\テスト
$ set RUBYLIB=.

C:\usr\sandbox\ruby\req\テスト
$ echo %RUBYLIB%
.

C:\usr\sandbox\ruby\req\テスト
$ ruby -v
ruby 2.3.3p222 (2016-11-21 revision 56859) [i386-mingw32]

C:\usr\sandbox\ruby\req\テスト
$ ruby a.rb
a
b

C:\usr\sandbox\ruby\req\テスト
$ ruby -E cp932:utf-8 a.rb
<internal:enc/prelude>:4:in `require': stack level too deep (SystemStackError)
        from <internal:enc/prelude>:4:in `<internal:enc/prelude>'

C:\usr\sandbox\ruby\req\テスト
$ set RUBYLIB=

C:\usr\sandbox\ruby\req\テスト
$ ruby -E cp932:utf-8 a.rb
a
b

なお、a.rb の require './b' を require 'b' と変更すると、
require が失敗します。

C:\usr\sandbox\ruby\req\テスト
$ type a2.rb
puts 'a'
require 'b'

C:\usr\sandbox\ruby\req\テスト
$ ruby -I. a2.rb
a
c:/programs/ruby/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require': cannot load such file -- b (LoadError)
        from c:/programs/ruby/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
        from a2.rb:2:in `<main>'

以前報告されている事例の別パターンのように思われます。

#3 [ruby-dev:50225] Updated by usa (Usaku NAKAMURA) 11 months ago

  • Backport changed from 2.2: UNKNOWN, 2.3: UNKNOWN, 2.4: UNKNOWN to 2.2: WONTFIX, 2.3: REQUIRED, 2.4: REQUIRED

あー、再現しました。ありがとうございます。

カレントディレクトに日本語を含む

これがすっぽ抜けてました。

あとは、RUBYLIB の場合だけで、-I などでは問題ないようですね。なるほど。

#4 Updated by usa (Usaku NAKAMURA) 11 months ago

  • Description updated (diff)

#5 [ruby-dev:50228] Updated by dogatana (Toshihiko Ichida) 11 months ago

あとは、RUBYLIB の場合だけで、-I などでは問題ないようですね。なるほど。

そうですね。
a2.rb で -I. を使用する場合だと、require が失敗するだけで、
いきなり終了する現象は出ません。

require が失敗するケースも対策されるとうれしいです。

#6 Updated by nobu (Nobuyoshi Nakada) 8 months ago

  • Status changed from Open to Closed

Applied in changeset trunk|r60743.


load.c: cwd encoding

  • load.c (rb_get_expanded_load_path): save cwd cache in OS path encoding, to get rid of unnecessary conversion and infinite loading when it needs encoding conversion. [Bug #13863]

#7 [ruby-dev:50500] Updated by nagachika (Tomoyuki Chikanaga) 4 months ago

  • Backport changed from 2.2: WONTFIX, 2.3: REQUIRED, 2.4: REQUIRED to 2.2: WONTFIX, 2.3: REQUIRED, 2.4: DONE

ruby_2_4 r62784 merged revision(s) 58745,58780,59040,60743.

#9 [ruby-dev:50502] Updated by nagachika (Tomoyuki Chikanaga) 4 months ago

  • Backport changed from 2.2: WONTFIX, 2.3: REQUIRED, 2.4: DONE to 2.2: WONTFIX, 2.3: REQUIRED, 2.4: REQUIRED

テストが通らなかったので ruby_2_4 へのバックポート r62797, r62784 は revert しました。

Also available in: Atom PDF