Skip to content

Fixed deprecation warnings for use of Chef::ShellOut#47

Closed
sroccaserra wants to merge 1 commit into
sous-chefs:masterfrom
sroccaserra:master
Closed

Fixed deprecation warnings for use of Chef::ShellOut#47
sroccaserra wants to merge 1 commit into
sous-chefs:masterfrom
sroccaserra:master

Conversation

@sroccaserra

@sroccaserra sroccaserra commented Apr 4, 2013

Copy link
Copy Markdown

Fixes COOK-2488 and COOK-2415.

See also pull request #37 and #46.

Cheers!

@sroccaserra sroccaserra mentioned this pull request Apr 4, 2013
Merged
@nviennot

nviennot commented Apr 11, 2013

Copy link
Copy Markdown

+1

1 similar comment
@dkador

dkador commented Apr 14, 2013

Copy link
Copy Markdown

+1

@jtimberman

jtimberman commented Apr 17, 2013

Copy link
Copy Markdown

Thank you for this pull request, however, I merged #46 - the include method doesn't appear to work in a recipe:

chef:recipe > include Chef::Mixin::ShellOut
NameError: Cannot find a resource for include on mac_os_x version 10.8.3
    from /Users/jtimberman/.rbenv/versions/1.9.3-p327/lib/ruby/gems/1.9.1/gems/chef-11.4.0/lib/chef/resource_platform_map.rb:126:in `get'
    from /Users/jtimberman/.rbenv/versions/1.9.3-p327/lib/ruby/gems/1.9.1/gems/chef-11.4.0/lib/chef/resource.rb:758:in `resource_for_platform'
    from /Users/jtimberman/.rbenv/versions/1.9.3-p327/lib/ruby/gems/1.9.1/gems/chef-11.4.0/lib/chef/resource.rb:775:in `resource_for_node'
    from /Users/jtimberman/.rbenv/versions/1.9.3-p327/lib/ruby/gems/1.9.1/gems/chef-11.4.0/lib/chef/dsl/recipe.rb:55:in `method_missing'
    from (irb#1):5
chef:recipe > Chef::Recipe.send(:include, Chef::Mixin::ShellOut)
 => Chef::Recipe

Using Mixlib::ShellOut.new instead of shell_out() in the recipe is fine.

@jtimberman jtimberman closed this Apr 17, 2013
@lock

lock Bot commented May 3, 2019

Copy link
Copy Markdown

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 May 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@sroccaserra @nviennot @dkador @jtimberman