Skip to content

Commit

Permalink
Completely remove tomcat::install_from_source
Browse files Browse the repository at this point in the history
This parameter is not used anymore and was deprecated more than 2 years
ago.  2 major versions of the module where released with this
deprecation.

When set, this parameter makes the catalog compilation fail with an
error message explaining that this parameter is deprecated.  However,
when set to false or undef, no error is raised, but the default
behavior is still to install from source…

Completely remove this parameter so that setting it to any value will
produce an error.
  • Loading branch information
smortex committed Sep 30, 2019
1 parent a3f92d1 commit 94bb104
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
6 changes: 0 additions & 6 deletions manifests/init.pp
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,6 @@
# Specifies a default user to run Tomcat as. Valid options: a string containing a valid username.
# @param group
# Specifies a default group to run Tomcat as. Valid options: a string containing a valid group name.
# @param install_from_source
# No longer available in the base class. Please use install_from_source on a specific tomcat::install declaration instead.
# @param purge_connectors
# Specifies whether to purge any unmanaged Connector elements that match defined protocol but have a different port from the configuration file by default.
# @param purge_realms
Expand All @@ -27,7 +25,6 @@
$catalina_home = '/opt/apache-tomcat',
$user = 'tomcat',
$group = 'tomcat',
$install_from_source = undef,
Boolean $purge_connectors = false,
Boolean $purge_realms = false,
Boolean $manage_user = true,
Expand All @@ -37,9 +34,6 @@
Boolean $manage_properties = true,
) {

if $install_from_source {
fail('install_from_source is no longer available in the base class. Please use install_from_source on a specific tomcat::install declaration instead.') # lint:ignore:140chars
}
case $::osfamily {
'windows','Solaris','Darwin': {
fail("Unsupported osfamily: ${::osfamily}")
Expand Down
1 change: 0 additions & 1 deletion spec/classes/tomcat_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
end
let :params do
{
install_from_source: false,
}
end

Expand Down

0 comments on commit 94bb104

Please sign in to comment.