Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issue 26 #27

Closed
wants to merge 1 commit into from
Closed

Fix issue 26 #27

wants to merge 1 commit into from

Conversation

harobed
Copy link

@harobed harobed commented Jul 11, 2013

Fix issue 26 : « Error if hostmanager configuration option missing in Vagrantfile. My suggest => consider as enable=false »

… Vagrantfile. My suggest => consider as enable=false »
@harobed
Copy link
Author

harobed commented Jul 19, 2013

bump

@smdahlen
Copy link
Contributor

@harobed Taking a look now. I prefer to keep the finalize method as that is the documented approach for Vagrant plugins. I assume this fails when you are running any command?

@harobed
Copy link
Author

harobed commented Jul 19, 2013

  1. I have "vagrant-hostmanager" installed on my box
  2. I execute vagrant up on Vagrantfile without "config.hostmanager.enabled" option
  3. => I've an error

@smdahlen
Copy link
Contributor

ok, thanks. I should have a fix within an hour or so published (and/or merge your suggestion).

@smdahlen
Copy link
Contributor

@harobed what version of vagrant are you running?

@harobed
Copy link
Author

harobed commented Jul 19, 2013

$ vagrant --version
Vagrant version 1.2.3

@smdahlen smdahlen closed this Jul 19, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants