From bfabbaf6ef2e23b006d5b5f0125d274bf15cd055 Mon Sep 17 00:00:00 2001 From: Jeremy Stanley Date: Mon, 10 Mar 2014 13:49:14 +0000 Subject: [PATCH] Explicitly install gettext on job workers Some providers include gettext preinstalled by default on their base images, but others do not. Make this consistent by explicitly installing it when not already present. Rackspace seems to have recently ceased including it on their Ubuntu Precise images, which precipitated this patch. Change-Id: Idb86c805e2fc56d885081876642bd9d3dbf58434 --- modules/jenkins/manifests/params.pp | 2 ++ modules/jenkins/manifests/slave.pp | 1 + 2 files changed, 3 insertions(+) diff --git a/modules/jenkins/manifests/params.pp b/modules/jenkins/manifests/params.pp index 50660e4088..77cd016900 100644 --- a/modules/jenkins/manifests/params.pp +++ b/modules/jenkins/manifests/params.pp @@ -27,6 +27,7 @@ class jenkins::params { # packages needed by document translation $gnome_doc_package = 'gnome-doc-utils' $libtidy_package = 'libtidy' + $gettext_package = 'gettext' # for keystone ldap auth integration $libsasl_dev = 'cyrus-sasl-devel' $nspr_dev_package = 'nspr-devel' @@ -110,6 +111,7 @@ class jenkins::params { # packages needed by document translation $gnome_doc_package = 'gnome-doc-utils' $libtidy_package = 'libtidy-0.99-0' + $gettext_package = 'gettext' # for keystone ldap auth integration $libsasl_dev = 'libsasl2-dev' $mysql_dev_package = 'libmysqlclient-dev' diff --git a/modules/jenkins/manifests/slave.pp b/modules/jenkins/manifests/slave.pp index a9d024b600..8ccf917306 100644 --- a/modules/jenkins/manifests/slave.pp +++ b/modules/jenkins/manifests/slave.pp @@ -39,6 +39,7 @@ class jenkins::slave( $::jenkins::params::docbook_xml_package, # for building openstack docs $::jenkins::params::docbook5_xml_package, # for building openstack docs $::jenkins::params::docbook5_xsl_package, # for building openstack docs + $::jenkins::params::gettext_package, # for msgfmt, used in translating manuals $::jenkins::params::gnome_doc_package, # for generating translation files for docs $::jenkins::params::graphviz_package, # for generating graphs in docs $::jenkins::params::firefox_package, # for selenium tests