jimmycuadra / vagrant-librarian-chef

*UNMAINTAINED* A Vagrant plugin to install Chef cookbooks using Librarian-Chef.
MIT License
78 stars 15 forks source link

Use '.' as the default for `cheffile_dir`. #6

Closed whilp closed 11 years ago

whilp commented 11 years ago

The existing default (nil) causes the following failure on Vagrantfiles that do not specify cheffile_dir:

$ vagrant -v
Vagrant version 1.1.5
$ vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
[default] Importing base box 'opscode-ubuntu-12.04'...
[default] Matching MAC address for NAT networking...
[default] Setting the name of the VM...
[default] Clearing any previously set forwarded ports...
[default] Installing Chef cookbooks with Librarian-Chef...
[default] Destroying VM and associated drives...
/Users/wcmaier/.vagrant.d/gems/gems/vagrant-librarian-chef-0.1.1/lib/vagrant-librarian-chef/action/librarian_chef.rb:20:in `join': can't convert nil into String (TypeError)
[...]
$ cat Vagrantfile 
#!/usr/bin/env ruby

Vagrant.configure('2') do |config|
  config.vm.hostname = 'ceilometer'
  config.vm.box = ENV['VAGRANT_BOX'] || 'opscode-ubuntu-12.04'
  config.vm.box_url = ENV['VAGRANT_BOX_URL'] || "https://opscode-vm.s3.amazonaws.com/vagrant/boxes/#{config.vm.box}.box"

  config.vm.provision :chef_solo do |chef|
    chef.run_list = [
      'recipe[ceilometer::default]'
    ]
  end
end
$ cat Cheffile
site 'http://community.opscode.com/api/v1'

cookbook 'ceilometer',
         :path => './'
jimmycuadra commented 11 years ago

Thanks! Released in v0.1.2.

whilp commented 11 years ago

Thank you!