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

Feature/windows support #154

Closed
wants to merge 11 commits into from
Closed

Conversation

calvn
Copy link

@calvn calvn commented Mar 25, 2015

Fixed feature/windows_support to be on a working state against master in addition to some chocolatey tweaks.

A lot of credit to @ramarnat for the initial #119 PR

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 40531ae on cleung2010:feature/windows_support into 76f7831 on johnbellone:master.

@ramarnat
Copy link

nice!

@johnbellone
Copy link
Contributor

@cleung2010 Is this ready to merge?

@calvn
Copy link
Author

calvn commented Apr 2, 2015

Yep!

@ghost
Copy link

ghost commented Apr 24, 2015

even though consul metadata has depend 'chcocolatey', my chef run fails as below
uninitialized constant Chef::Node::ChocolateyHelpers

Looks like it can not find the resource ChocolateyHelpers defined in the libraries on chocolatey cookbook

@ghost
Copy link

ghost commented Apr 24, 2015

It was because of my chef version upgraded from 11.6.2 to 11.18.6 and the problem went away.

Also I used #{ENV.keys.grep(/^ChocolateyInstall$/i)} instead of calling the helper function in 11.6 and that solved the problem

Tested the install_windows recipe and it works the only problems were chocolatey ones due to chef version, 11.18.6 worked without a problem

@johnbellone johnbellone added this to the Road to 1.0.0 milestone May 1, 2015
@johnbellone johnbellone self-assigned this May 1, 2015
@johnbellone
Copy link
Contributor

Please rebase!

@ghost ghost mentioned this pull request Jun 8, 2015
@calvn
Copy link
Author

calvn commented Jun 13, 2015

@johnbellone I fixed conflicts with master, should be good to merge now

@johnbellone
Copy link
Contributor

We are going to be merging #126 soon. Please take a look at that as it changes the installation for clients.

@lock
Copy link

lock bot commented Apr 25, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants