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

support of windows guests #59

Closed
wants to merge 2 commits into from
Closed

Conversation

jarig
Copy link

@jarig jarig commented Jan 5, 2014

Windows guest support.
Unit tests and acceptance tests passed on Win8 and Win2k8(tried only with virtualbox provider).

@schisamo
Copy link
Contributor

schisamo commented Jan 5, 2014

@jarig This is awesome! I'm AFK today but will take a deeper review ASAP.

@schisamo
Copy link
Contributor

schisamo commented Jan 5, 2014

@jarig any chance you can take a look at and address the Travis CI failures also?

@jarig
Copy link
Author

jarig commented Jan 6, 2014

rake test:unit works on my machine, but fails to start in Travis, that seems to be following Vagrant issue which appears on 64bit linux machines:
hashicorp/vagrant#2229

@schisamo
Copy link
Contributor

schisamo commented Jan 6, 2014

So finally figured what the issue is. Log4r 1.1.11 was released (and yanked). This version changed the arity of it's constructor in a patch release which broke core vagrant. For some reason Travis is still pulling down log4r 1.1.11 though!

Anyway thanks for fixing some of the Rubocop errors. I'm going to pull your changes into a branch a do a little more cleanup to fix the rest of the Rubocop issues and then merge.

@schisamo schisamo mentioned this pull request Jan 7, 2014
@schisamo
Copy link
Contributor

schisamo commented Jan 7, 2014

Closing this as I opened #60 which contains these changes plus a few more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants