From 0307cc047f253a18a36a23cb128b862e113bf414 Mon Sep 17 00:00:00 2001 From: varac Date: Thu, 5 Nov 2015 13:36:01 +0100 Subject: [bug] [jessie] Don't specify ruby versions because ruby-1.9.3 is not available on jessie. - Related: #6920 --- puppet/modules/site_config/manifests/ruby/dev.pp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'puppet/modules/site_config/manifests/ruby/dev.pp') diff --git a/puppet/modules/site_config/manifests/ruby/dev.pp b/puppet/modules/site_config/manifests/ruby/dev.pp index 3ea6ca96..e6eb2f8a 100644 --- a/puppet/modules/site_config/manifests/ruby/dev.pp +++ b/puppet/modules/site_config/manifests/ruby/dev.pp @@ -1,6 +1,6 @@ +# install ruby dev packages needed for building some gems class site_config::ruby::dev inherits site_config::ruby { Class['::ruby'] { - ruby_version => '1.9.3', install_dev => true } # building gems locally probably requires build-essential and gcc: -- cgit v1.2.3 From 38e1003186ca194c75b87fdb8898304556d89a7a Mon Sep 17 00:00:00 2001 From: varac Date: Fri, 22 Jan 2016 23:52:58 +0100 Subject: [bug] Use ruby::devel to install ruby-dev deb Ruby itself is a parameterized class, and parameters cannot get overridden (see https://projects.puppetlabs.com/issues/9259). The webapp node didn't install the ruby-dev package (we never noticed because our vagrant images as probably other debian images had ruby-dev preinstalled). We now use the ruby::devel class to install ruby-dev. - Tested: [citest-jessie] - Resolves: #7838 --- puppet/modules/site_config/manifests/ruby/dev.pp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'puppet/modules/site_config/manifests/ruby/dev.pp') diff --git a/puppet/modules/site_config/manifests/ruby/dev.pp b/puppet/modules/site_config/manifests/ruby/dev.pp index e6eb2f8a..2b0b106d 100644 --- a/puppet/modules/site_config/manifests/ruby/dev.pp +++ b/puppet/modules/site_config/manifests/ruby/dev.pp @@ -1,8 +1,8 @@ # install ruby dev packages needed for building some gems -class site_config::ruby::dev inherits site_config::ruby { - Class['::ruby'] { - install_dev => true - } +class site_config::ruby::dev { + include site_config::ruby + include ::ruby::devel + # building gems locally probably requires build-essential and gcc: include site_config::packages::build_essential } -- cgit v1.2.3