haskal
/
vagrant
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Check name config option with respond_to? 

Prior to this commit, the check used to look for the config option
`name` in a provisioner config would accidentally create a "DummyConfig"
based on how vagrant handles missing config options. This commit fixes
that by instead using the `respond_to?` method to check for the
existance of the config option name.
This commit is contained in:
Brian Cain 2020-01-07 10:29:39 -08:00
parent dd5d73e8a9
commit 6855dd8c74
No known key found for this signature in database
GPG Key ID: 9FC4639B2E4510A0
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/vagrant/action/builtin/mixin_provisioners.rb
View File

@ -35,7 +35,8 @@ module Vagrant
# Note: `name` is set to a symbol, since it is converted to one via #Config::VM.provision # Note: `name` is set to a symbol, since it is converted to one via #Config::VM.provision
provisioner_name = provisioner.name provisioner_name = provisioner.name
if !provisioner_name if !provisioner_name
if provisioner.config.name if provisioner.config.respond_to?(:name) &&
provisioner.config.name
provisioner_name = provisioner.config.name.to_sym provisioner_name = provisioner.config.name.to_sym
end end
else else