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

Using = instead of == #54

Closed
bradphelan opened this issue Nov 28, 2011 · 3 comments · Fixed by #55
Closed

Using = instead of == #54

bradphelan opened this issue Nov 28, 2011 · 3 comments · Fixed by #55
Labels
Bug Something isn't working

Comments

@bradphelan
Copy link
Contributor


/Users/bradphelan/workspace/mysugr/mysugrapp/vendor/cookbooks/rvm/libraries/rvm_rubygems_package.rb

123 def install_package(name, version)
124 # ensure each ruby is installed and gemset exists
125 ruby_strings.each do |rubie|
126 next if rubie = 'system'
127 e = rvm_environment rubie do
128 user new_resource.user if new_user.respond_to?("user")
129 action :nothing

line 126 should be

if rubie == 'system'

@bradphelan bradphelan mentioned this issue Nov 28, 2011
@fnichol
Copy link
Contributor

fnichol commented Nov 28, 2011

You're right, that looks wrong to me!

@fnichol
Copy link
Contributor

fnichol commented Nov 28, 2011

Thanks for the find, you helped uncover an additional method_missing bug!

phlipper pushed a commit to newleaders/chef-rvm that referenced this issue Feb 1, 2012
@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
Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants