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

Ability to pass compile options, pkg install to Rvm_system_ruby #68

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Ability to pass compile options, pkg install to Rvm_system_ruby #68

wants to merge 3 commits into from

Conversation

rwilcox
Copy link

@rwilcox rwilcox commented Mar 7, 2013

I've added support for installing rvm packages via the Rvm_system_ruby type, and also added the ability to pass compile time options.

I implemented this because I found that the OpenSSL version on my OS was too old for RubyGems 2.0 / Ruby 2.0. I followed the instructions in a gist (https://gist.github.com/jfirebaugh/4007524) to use RVM's openssl package and that worked! But I wanted to automate that, so I made some modifcations.

Anyway, hope this helps! Let me know if I can change anything, etc etc.

rwilcox added 3 commits March 7, 2013 16:53
Conflicts:
	lib/puppet/provider/rvm_system_ruby/rvm_system_ruby.rb
	lib/puppet/type/rvm_system_ruby.rb
@blt04
Copy link
Owner

blt04 commented Mar 9, 2013

Looks nice. But what if someone needs the OpenSSL package for multiple rubies? In that case it looks like OpenSSL is installed twice. Maybe we need to create an rvm_package type/provider?

@rwilcox
Copy link
Author

rwilcox commented Mar 10, 2013

But what if someone needs the OpenSSL package for multiple rubies?

Yeah, that's a problem I solved in an in-elegant way. My, Ryan-doesn't-know-Puppet-all-that-well solution was to have one RVM_System_Ruby have a pkg declaration, then have my other Ruby require the first Ruby. Thus I know I'll have the OpenSSL package and I'll only get it once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants