今日は rubygems と bundler の更新の cherry-pick で変更がたくさんありました。本体のほうは Refinements の不具合修正? が 1つありました。
標準添付ライブラリ rdoc でファイルに対して Marshal.dump で出力するのに一旦文字列オブジェクトにしてから書き込んでたので Marshal.dump の第2引数に IO を渡す方法を使うようにリファクタリングしています。
rubygems のテストでただの require のかわりに assert_require という assertion を使うようにしています。こんな assertion メソッドがあるのか。
rubygems の require のテストで loaded_spec_names のチェックも追加しています。
rubygems が上書きする require で $LOAD_PATH 内の gem のパスからファイルを探す通常の探索(のように思える)をその他の処理の前に行うようにしています。 -I オプションによる指定が default gem より優先して効くようにするためらしいです。なんか遅くなりそうだけど…。
00cd5d74cecb6aa4a184e57e2b8246ef6e45d458 の追加修正。 rubygems が古くて Gem.load_path_insert_index が nil だったら $LOAD_PATH を全部調べるように終端を -1 にするようにしています。ますます遅くなりそう…(まあ古い場合だけみたいだけど)。
00cd5d74cecb6aa4a184e57e2b8246ef6e45d458 のさらに再修正。ファイルパス探索時に File.expand_path したものを使うようにしています。コミットログによると JRuby での不具合修正らしい。
Gem::Command#show_lookup_failure の第4引数 domain を削除してかわりに省略可能引数 suppress_suggestions を追加し実体に名前と渡す内容をあわせるようにリファクタリングしています。
lib/rubygems.rb の Gem.register_default_spec からファイルの suffix の正規表現の変数をメソッド Gem.suffix_regexp として切り出すようにしています。
rubygems の再定義する require で同じファイルを重複してロードしてしまう場合があった不具合を修正しているようです。
1ac6890bd05a48e385f564d212765a3a60899659 の再修正。よくわかりませんが default gem の require した時に @path_to_default_spec_map から消すのはやめて、load 済みかどうかのチェックのほうで重複を回避するようにしてるみたいです。
これもよくわかりませんが rubygems の bundler との協調のためのコードで Gemfile.lock に書かれる bundler のバージョンより実際の bundler のバージョンを優先するようにしているっぽいです。
5998012a0c7244fe217c2d4f494e50f32ec85d29 の再修正。指定の bundler のバージョンがみつからない時に nil を返すように変更。
rubygems の Gem::RubyGemsVersion と Gem::ConfigMap という定数を deprecate にしています。
rubygems のテストで $LOAD_PATH にソースディレクトリの lib が含まれてたらそれを末尾に移動するようにしています。default gem のロードの優先度とかが変わっちゃうからかな。
00cd5d74cecb6aa4a184e57e2b8246ef6e45d458 の変更で rubygems で再定義された require で tainted な文字列を require に渡す可能性があったのを修正しています。
00cd5d74cecb6aa4a184e57e2b8246ef6e45d458 の rubygems の再定義する require での default gems より -I オプションを優先するための変更とそれに追随する変更を revert しています。
.github/workflows/ の GitHub Actions の設定の fail-fast というオプションを false に設定しています。一部のタスク? ジョブ? が失敗しても続きのテストも実行したいため。
Object#method_missing の rdoc 用コメントのサンプルコードを修正しています。メソッドの引数をちゃんと受け取るのと処理できない時に super を呼んでぶようにする作法を示しています。
Refinements で Module#using で Refinements を適用している Module 内でさらに Module#refine で refinements のブロックを開くと、その中では外のスコープの using が効かないという不具合を修正しています。これ不具合なのかな…。まあ Refinements は lexical に効くはずだから効いたほうがいいのか。けどなんとなくサンプルをひとめ見た瞬間は、こっちが正しいような気がしてしまったけど…。 [ruby-core:94382] [Bug #16107]
11a9f7ab9431b0f361e43b4ac2bd6ee44827d88b の追加で Refinements 関連の method entry 探索を search_refined_method() という関数に切り出しています。 [ruby-core:94382] [Bug #16107]
ea496e9fb2c930ac0c5ff8c12c4a10fac0ec3c7b の make check 追加に追随して continue-on-error: true の設定が make check についてしまわないよう除外の条件を復活させています。
bundler の deprecation 警告の typo 修正。
Fiber#resume の rdoc 用コメントのサンプルで fiber.resume に渡す引数の数値を変更しています。なんかコミットログにいろいろ書いてあるけど、元のサンプルだと最後(にエラーにならずに呼べる) Fiber#resume の戻り値がその前の Fiber.yield に渡した値と関係あるようにみえてしまって混乱するから、みたいです。うーん、そうかな。そうかも。 https://github.com/ruby/ruby/pull/2170
75d29db8f965893bb6ab38b9008abc80cdda246e で revert した 00cd5d74cecb6aa4a184e57e2b8246ef6e45d458 はじめとする rubygems の再定義する require で default gems より -I の指定を優先するためのパスを再度追加しています。特に重大な差分はないのでなにか追加修正を一緒にしたというわけではなさそう。
rubyspec の require のテストで $LOAD_PATH にパスを追加する時に先頭でなく最後に追加するようにしています。 rubygems の require 関連の変更の影響でしょうね。
dd16f8524c44f8802e634af2577fe5a149cd740e で復活させた rubygems による require 実装でディレクトリパスが symbolic link だった時の対処を追加しています。
dd16f8524c44f8802e634af2577fe5a149cd740e で復活させた rubygems の再定義 require で先に $LOAD_PATH 内にみつけたファイルをオリジナルの require に渡した時になぜか LoadError が発生した場合にも対処して残りの処理に fallback させるように例外の捕捉と RUBYGEMS_ACTIVATION_MONITOR のロック再取得するようにしています。
ruby 本体の require のテストで普通に require と書くと rubygems の再定義した Kernel#require が呼ばれてしまってうまくテストできないことがあるので、Kernel.send(:require, ...) でオリジナルのメソッドを呼ぶようにしています。あーなるほど Kernel#require は上書きされるけど特異メソッドの Kernel.require のほうはオリジナルのままなのか。
dd16f8524c44f8802e634af2577fe5a149cd740e の rubygems 版の require の対処で Gem.load_path_insert_index が nil の時の対処はやっぱり消しています。これが nil になるのは 1.9 より前らしいので、さすがにそれはもう考慮しないでいいということで。
同じく dd16f8524c44f8802e634af2577fe5a149cd740e で復活させた rubygems 版 require の処理追加のところに意図の説明と rubygems の pull request へのリンクをコメントで追加しています。
260ef51a73c067599826c8ab110c53994c1b6226 で revert した古い ruby 用の Gem.load_path_insert_index が nil を返すケースの対応はやっぱり復活させています。 2.4/2.5 の安定版で壊れるとのこと。
182a408c2c4113eb316c2a87e35880144afb4498 で Proc#to_s および #inspect のファイル名/行番号の表示の前の "@" を削って空白にした変更で rubygems でこれに依存してた部分の修正もしてましたが、古い ruby への対応のためどっちでも動くように再修正しています。
bundler のテストのヘルパーメソッド bundler_path が lib というメソッドとまったく同じ内容だったので削除して lib を使うようにしています。しかし lib ってずいぶん思いきった名前だな…。
ありそうだなーと思ったらやっぱり bundler のテストのヘルパーメソッド内で lib っていうローカル変数を使ってるところがあったので削除して、内容的には同じだったので lib メソッドをそのまま使うようにしています。
話題の lib メソッドを spec/bundler/support/helpers.rb から spec/bundler/support/path.rb に移動しています。
bundler のテストの let で bundler_dir というのを定義してたのを同じ内容のヘルパーメソッド root を使うようにしています。
bundler のテストでローカル変数への代入を繰り返してたところをメソッドチェインに書き換え。
bundler のテストで let で setup_path というのを宣言してたのを、これもヘルパーメソッド lib を使って各々の場所で書くようにしています。
bundler のテストでコマンドとしての bundle ファイルの位置を ruby リポジトリ内では libexec/bundle にするように変更しています。
63575409458f6b7b1bdaf038a562bc3f011e231e の続きと思われますが bundler のテストで bundle コマンドの位置を特定するのに ruby リポジトリ内かどうかの分岐をヘルパーメソッド bindir にまかせるようにリファクタリングしています。
bundler のテストで ruby 実行時に -rrubygems オプションをつけてたのは、最近の ruby だとデフォルトで rubygems 有効で不要なので削除しています。
2e6509498c1daf3335ac6343bf59441d42011b64 の続き。そもそもオプションの参照が不要ということで条件分岐ごと削除しています。
bundler のテストで gem_command! というヘルパーメソッドを使うようにするリファクタリング。
bundler のテストのヘルパーメソッド gem_bin というのを切り出しています。
91c63828f7b841df6b7b124acb752c0a3bca66c0 で切り出した gem_bin で gem コマンドに --backtrace オプションを追加するようにしています。
さらに bundler のテストのヘルパーメソッド gem_command でも切り出した gem_bin を流用するようにしています。
bundler の bundle gem のテストで bundler.gem は不要なので用意しないようにしています。
bundler のファイルリストについてのテストで変数名の改名。
b43f4bd2184989c5f47ee539e583c785dfadb8d0 で改名した tracked_files という変数を bundler のテストのヘルパーメソッドとして定義するように切り出しています。
同じく bundler のテストで変数名の改名。これもヘルパーメソッドにする布石みたいです。
ということで 19dabfbe574f9a8298054eb417d299ec5fc18412 で改名した変数を bundler のテストのヘルパーメソッドとして切り出し。
bundler のテストのヘルパーメソッド install_gems でファイルパスの生成に + のかわりに Pathname#join を使うようにリファクタリング。
bundler のテストのヘルパーメソッドの install_gems には install_gem という alias も残してあったのですが、最後の呼び出し元も改名して alias は削除しています。
と思ったら 20af44c27727333b4cd3eef9d5b18daaa0445f0e で削除した alias の install_gem は install_gem から呼び出す 1つの gem のインストールの処理のメソッドとして定義しています。
bundler のテストのヘルパーメソッドで Spec::Path という定数参照をしていたところを単に Path を使うようにしています。なんか bundler のテストは省記述指向だな。
bundler のテストのヘルパーメソッド gem_command! の第2引数に渡す Pathname と to_s してから渡してるところはそのまま渡すようにしています。gem_command! の中でも to_s しているため。
bundler のテストで FileUtils.rm に渡す引数でも to_s してるところがありましたが Pathname も受け付けるのでそのまま渡すようにしています。
同じ bundler のテストでファイル削除に FileUtils.rm のかわりに絶対パスにした上で Pathname#rmtree を使うようにしています。
でもってその絶対パス化した Pathname をローカル変数に一度代入しておくようにしています。
bundler のテストで Dir.chdir でカレントディレクトリを移動する範囲を狭めて必要なタイミングでのみ移動するようにしています。
bundler のテストで bundler.gemspec を作ってテストする処理をヘルパーメソッド with_root_gemspec として切り出しています。
09455301ef81bc6207e6d3ba1034c35851575e18 で切り出した with_root_gemspec でさらに gemspec ファイルパスを変数に入れて使いまわすようにリファクタリングしています。
bundler のテストのヘルパーメソッド with_built_bundler というのも切り出しています。
bundler のテストで gemspec ファイルが置かれるディレクトリを取得するヘルパーメソッド gemspec_dir を追加しています。それは root とは違うのか…と思ったけど ruby リポジトリ内の場合はサブディレクトリ内になるんですね。
bundler のテストで $LOAD_PATH を弄った後で復帰する処理は JRuby では実行しないようにしています。 JRuby の $LOAD_PATH は replace できないのかな。
標準添付ライブラリ rdoc のテストのディレクトリ構成の変更。
rdoc の RDoc::Markup::FormatterTestCase と RDoc::Markup::TextFormatterTestCase の autoload 設定の削除。
rdoc の gemspec ファイルの spec.files から削除されたファイルを消しています。
rdoc のテストのうち Travis-CI で skip するようにしていたテストの skip をとりやめています。ファイル構成の変更で直ったのかな。
rdoc の gemspec ファイルの spec.files から .travis.yml を削除しています。