Skip to content
This repository has been archived by the owner. It is now read-only.

HEADS UP! Bindler might break on Vagrant 1.4 #26

Closed
fgrehm opened this issue Dec 4, 2013 · 8 comments
Closed

HEADS UP! Bindler might break on Vagrant 1.4 #26

fgrehm opened this issue Dec 4, 2013 · 8 comments

Comments

@fgrehm
Copy link
Owner

@fgrehm fgrehm commented Dec 4, 2013

I suspect this change might break things over here

/cc @patcon @ipwnstuff

@mitchellh
Copy link

@mitchellh mitchellh commented Dec 4, 2013

Since bindler is so dependent on some internal Vagrant state, I recommend putting this sort of check in the plugin.rb file:

if Vagrant::VERSION >= "1.4.0"
    puts "NOPE"
end

Sort of thing.

@fgrehm
Copy link
Owner Author

@fgrehm fgrehm commented Dec 4, 2013

@mitchellh You beat me :P I was just spreading the word and was about to mention that over here. We really need to add a check for compatible vagrant versions over here to prevent that kind of stuff :)

@erran
Copy link
Collaborator

@erran erran commented Dec 4, 2013

I think it's handled well in vagrant-berkshelf see the install action they've defined. If you agree I'm up for creating a PR.

@fgrehm
Copy link
Owner Author

@fgrehm fgrehm commented Dec 5, 2013

@ipwnstuff 👍 be my guest 😃

@fgrehm
Copy link
Owner Author

@fgrehm fgrehm commented Jan 27, 2014

According to @rogeriopradoj on GH-29 the plugin is really broken, let's close this issue :-)

@fgrehm fgrehm closed this Jan 27, 2014
@rogeriopradoj
Copy link

@rogeriopradoj rogeriopradoj commented Jan 27, 2014

Hello, @fgrehm , trick question: are you using Bindler in your current setup? If positive, which versions (Vagrant, Os, provider etc.)?

;-)

@fgrehm
Copy link
Owner Author

@fgrehm fgrehm commented Jan 27, 2014

Not really, as I said on other issues I'm no longer using it and I'm stepping out of it for a while (but @erran and @patcon are both collaborators ;)

@rogeriopradoj
Copy link

@rogeriopradoj rogeriopradoj commented Jan 27, 2014

;-) . Firstly, congrats for your work around Vagrant ecosystem, and for gaining commit permission in official repo because of your great job. I really hope to meet you in person someday (maybe at Campus Party)?

See ya!

erran added a commit that referenced this issue Jan 27, 2014
Add Vagrant version constraints and checks (Fixes #26)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.