daptiv / dotnetframework

Chef cookbook for installing and configuring .NET framework 4.x
Apache License 2.0
8 stars 20 forks source link

Remove dependency upon deprecated windows_reboot resource #35

Closed sneal closed 6 years ago

tlopo commented 7 years ago

Windows_reboot was completely removed since 2016.09.07 version 2.0.0 we can see the changelog so we no longer get a warning, the cookbook simply fails now:

==> windows_consul: ================================================================================
==> windows_consul: 
==> windows_consul: NoMethodError
==> windows_consul: -------------
==> windows_consul: No resource or method named `windows_reboot' for `Chef::Recipe "default"'
==> windows_consul: 
==> windows_consul: Cookbook Trace:
==> windows_consul: ---------------
==> windows_consul:   C:/chef/cache/cookbooks/dotnetframework/recipes/default.rb:27:in `from_file'
sgtcoolguy commented 7 years ago

Any chance of a new supermarket release with this fix in it? I'm pointing at this repo for the fix done on this issue in my Berksfile, but it'd be nice to have a working version out on the Chef supermarket.

sofam commented 7 years ago

Could you release a new version to Chef supermarket with this fix in it?

heathsnow commented 6 years ago

https://github.com/daptiv/dotnetframework/pull/50

I'll upload to Supermarket after the PR is reviewed.