diff options
| author | Gabriel Filion <gabster@lelutin.ca> | 2015-12-04 14:29:36 -0500 |
|---|---|---|
| committer | Gabriel Filion <gabster@lelutin.ca> | 2015-12-04 14:35:27 -0500 |
| commit | 2942cd0dd88ec3a1d38197d148af9952a397b67c (patch) | |
| tree | 5f9d6a17cebc83cc1ae199f116cc281c753b7486 /manifests/init.pp | |
| parent | 4d3b720308964c027674bb08c49f142b4684ff58 (diff) | |
| download | puppet-apt-2942cd0dd88ec3a1d38197d148af9952a397b67c.tar.gz puppet-apt-2942cd0dd88ec3a1d38197d148af9952a397b67c.tar.bz2 | |
remove requirement on lsb package for sources.list file
Managing requirements for installing the lsb package has proven over
time to make no sense. The best approach to this is to require
lsb-release to be installed alongside puppet, since otherwise there are
so much facts that get no value during the run and you end up needing to
run puppet twice to get the real end result.
Also, since we're not including a class that is actually installing the
'lsb' package, that require line makes it so that including the apt
module doesn't work, and there's no documentation in the README about
needing to provide a package{'lsb':} resource with the apt class.
Because of all that, it makes more sense to just get rid of that require
line and mark lsb as a pre-requirement in the README file.
Diffstat (limited to 'manifests/init.pp')
| -rw-r--r-- | manifests/init.pp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/manifests/init.pp b/manifests/init.pp index 2660612..6732ade 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -50,7 +50,6 @@ class apt( # additional sources should be included via the apt::sources_list define '/etc/apt/sources.list': content => $sources_content, - require => Package['lsb'], notify => Exec['refresh_apt'], owner => root, group => 0, |