Skip to content

Commit

Permalink
Merge pull request voxpupuli#278 from fraenki/cond_apt_class
Browse files Browse the repository at this point in the history
respect $manage_repos, do not include ::apt if set to false
  • Loading branch information
dhoppe authored Dec 29, 2016
2 parents 7984ab2 + a981df3 commit 4c1223a
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion manifests/packages.pp
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
#
class php::packages (
$ensure = $::php::ensure,
$manage_repos = $::php::manage_repos,
$names_to_prefix = prefix(
$::php::params::common_package_suffixes, $::php::package_prefix # lint:ignore:parameter_documentation
),
Expand All @@ -30,7 +31,9 @@

$real_names = union($names, $names_to_prefix)
if $::osfamily == 'debian' {
include ::apt
if $manage_repos {
include ::apt
}
package { $real_names:
ensure => $ensure,
require => Class['::apt::update'],
Expand Down

0 comments on commit 4c1223a

Please sign in to comment.