Skip to content
This repository has been archived by the owner on Nov 6, 2018. It is now read-only.

Fix incompatibility with Chef 13 #25

Merged
merged 1 commit into from
May 22, 2017

Conversation

hartmantis
Copy link
Contributor

Per the Chef 13 release notes
LWRP resources no longer get constant names, resulting in this:

[2017-04-17T11:50:32-07:00] ERROR: mac_os_x_userdefaults[com.apple.dock autohide] (my-dev-machine::default line 120) had an error: NameError: uninitialized constant Chef::Resource::MacOsXUserdefaults
Did you mean?  Chef::Resource::MacosxService
Did you mean?  Chef::Resource::MacosxService
[2017-04-17T11:50:32-07:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

Signed-off-by: Jonathan Hartman j@p4nt5.com

Per the Chef 13 [release notes](https://docs.chef.io/release_notes.html)
LWRP resources no longer get constant names, resulting in this:

```
[2017-04-17T11:50:32-07:00] ERROR: mac_os_x_userdefaults[com.apple.dock autohide] (my-dev-machine::default line 120) had an error: NameError: uninitialized constant Chef::Resource::MacOsXUserdefaults
Did you mean?  Chef::Resource::MacosxService
Did you mean?  Chef::Resource::MacosxService
[2017-04-17T11:50:32-07:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
```

Signed-off-by: Jonathan Hartman <j@p4nt5.com>
@damacus damacus merged commit 3ec219a into Sous-Chefs-Boneyard:master May 22, 2017
tkalus pushed a commit to NunaInc/harden_macos that referenced this pull request May 23, 2017
Note: This cookbook works for Chef 13, but you must have this PR merged into
your mac_os_x cookbook: Sous-Chefs-Boneyard/mac_os_x#25.
This also fixes some line spacing issues on some of the commands.

- Change unit tests to debian
- Commented out tests until we can fix them
- Updated for foodcritic rules 66 and 67
@hartmantis hartmantis deleted the jdh-fix-for-chef-13 branch October 3, 2017 17:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants