diff options
| author | k0kubun <k0kubun@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2018-01-27 01:59:08 +0000 |
|---|---|---|
| committer | k0kubun <k0kubun@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2018-01-27 01:59:08 +0000 |
| commit | be27efdb83aadd82f0d7573f5b547f57cad472f5 (patch) | |
| tree | b55148b0985f3aa854bfe9949bb571f178a5c256 /tool/ruby_vm/controllers | |
| parent | d678c274aa747323846245ecc8225bb7ee1de04f (diff) | |
| download | ruby-be27efdb83aadd82f0d7573f5b547f57cad472f5.tar.gz | |
dumper.rb: stop getting --destdir option value
via current directory. It's kind of a global state and fragile to
implementation changes in other places, and how the `Dir.getwd` is set
is not obvious from just reading around `RubyVM::Dumper#initialize` if
it depends on the global state.
tool/ruby_vm/controllers/application_controller.rb: explicitly pass
destdir to RubyVM::Dumper.
tool/ruby_vm/scripts/insns2vm.rb: explicitly pass destdir parsed from
optparse.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@62064 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'tool/ruby_vm/controllers')
| -rw-r--r-- | tool/ruby_vm/controllers/application_controller.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tool/ruby_vm/controllers/application_controller.rb b/tool/ruby_vm/controllers/application_controller.rb index bb86873d25..65783513e7 100644 --- a/tool/ruby_vm/controllers/application_controller.rb +++ b/tool/ruby_vm/controllers/application_controller.rb @@ -16,9 +16,10 @@ require_relative '../models/typemap' require_relative '../loaders/vm_opts_h' class ApplicationController - def generate i + def generate i, destdir path = Pathname.new i - dumper = RubyVM::Dumper.new i + dst = Pathname.new(destdir).join(i) + dumper = RubyVM::Dumper.new dst return [path, dumper] end end |