-
Notifications
You must be signed in to change notification settings - Fork 6
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
bump low level dependencies to current main releases #8
Conversation
Thanks for the PR. How about the following compromise:
edited ^^^ I'm worried that other cookbooks that pessimistic lock to these older major versions would be broken. The windows one in particular would cause issues with Chef 11. |
I am not aware of many people using Chef 11 anymore, perhaps bump the cookbook to version 2.x to signify the large jump in low level dependencies? |
Ok, sold! Please remove the windows cookbook dependency (because it should no longer be needed in Chef 12) and add |
Ok give me till Monday/Tuesday to do this. |
No problem. I think you can ditch all of the cookbook dependencies. Apt_repository was released in 12.9 Chef chef/chef#4782 and yum_repository is also available in Chef as well. There is even chefspec examples: https://github.com/sethvargo/chefspec/tree/master/examples. |
After doing some research, require Chef 12.6+ (What Chef recommends for its windows package resource), drop the windows cookbook, but keep both the apt and yum cookbooks. Yum_repository was introduced in Chef 12.14, but it is not a stable chefdk release yet. |
I'm afraid I lead you astray, windows_package should remain windows package. |
apt ~> 4.0 yum ~> 3.0 Require Chef >= 12.6
Updated |
2.0 released, thanks again for the PR. |
apt ~> 4.0
yum ~> 3.0
windows ~> 2.0
These dependencies are woefully out of date and creating havoc on convergence right now.