Merge branch 'comment/master/jre_vs_jdk'
* comment/master/jre_vs_jdk: Change jre and jdk booleans to a single string
This commit is contained in:
@@ -12,33 +12,31 @@
|
|||||||
#
|
#
|
||||||
# [Remember: No empty lines between comments and class definition]
|
# [Remember: No empty lines between comments and class definition]
|
||||||
class java(
|
class java(
|
||||||
$jre = false,
|
$distribution = 'jdk',
|
||||||
$jdk = true,
|
$version = 'installed'
|
||||||
$version='1.6.0_25-fcs'
|
|
||||||
) {
|
) {
|
||||||
|
|
||||||
# Cannot pass anonymous arrays to functions in 2.6.8
|
# Cannot pass anonymous arrays to functions in 2.6.8
|
||||||
$v_true_false = [ '^true$', '^false$' ]
|
$v_distribution = [ '^jre$', '^jdk$' ]
|
||||||
# Must compare string values, not booleans.
|
# Must compare string values, not booleans.
|
||||||
validate_re("$jre", $v_true_false)
|
|
||||||
validate_re("$jdk", $v_true_false)
|
|
||||||
validate_re($version, '^[._0-9a-zA-Z:-]+$')
|
validate_re($version, '^[._0-9a-zA-Z:-]+$')
|
||||||
|
validate_re($distribution, $v_distribution)
|
||||||
|
|
||||||
$jre_real = $jre
|
$version_real = $version
|
||||||
$jdk_real = $jdk
|
$distribution_real = $distribution
|
||||||
$version_real = $version
|
|
||||||
|
|
||||||
if $jre_real {
|
case $distribution_real {
|
||||||
class { 'java::jre_package':
|
jre: {
|
||||||
version => $version_real,
|
class { 'java::jre_package':
|
||||||
stage => 'runtime',
|
version => $version_real,
|
||||||
|
stage => 'runtime',
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
jdk: {
|
||||||
|
class { 'java::jdk_package':
|
||||||
if $jdk_real {
|
version => $version_real,
|
||||||
class { 'java::jdk_package':
|
stage => 'runtime',
|
||||||
version => $version_real,
|
}
|
||||||
stage => 'runtime',
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user