diff options
author | Gabriel Filion <lelutin@gmail.com> | 2012-04-09 01:33:13 -0400 |
---|---|---|
committer | Gabriel Filion <lelutin@gmail.com> | 2012-04-09 01:38:30 -0400 |
commit | 4d0ddd005795592f56496d8d39dc37ee86fac038 (patch) | |
tree | 209b2fe420c7ddb258369668149c7f82637348c2 /manifests/init.pp | |
parent | 66433e665e611f305b0f65c61558a0ec8232da00 (diff) |
Refactor: manage .d directories when using sources_list and apt_conf
The .d directories are only managed by the main 'apt' class. However,
both 'sources_list' and 'apt_conf' defines depend on those directories.
So in practice, the defines have an implicit need for those directories
to be somehow managed.
Let's turn this into an explicit relation, and include the directories
in the defines.
This makes it possible to use both defines without having to include the
main 'apt' class. (maybe when using puppet apply?)
Signed-off-by: Gabriel Filion <lelutin@gmail.com>
Diffstat (limited to 'manifests/init.pp')
-rw-r--r-- | manifests/init.pp | 16 |
1 files changed, 1 insertions, 15 deletions
diff --git a/manifests/init.pp b/manifests/init.pp index b5be91f..0d58c61 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -128,21 +128,7 @@ class apt { require => [ Apt_conf['02show_upgraded'], Apt_conf['03clean'] ]; } - # watch .d directories and ensure they are present - file { "/etc/apt/apt.conf.d": ensure => directory, checksum => mtime; } - file { "/etc/apt/sources.list.d": - ensure => directory, - checksum => mtime, - notify => Exec['refresh_apt'], - } - - exec { - # "&& sleep 1" is workaround for older(?) clients - 'refresh_apt': - command => '/usr/bin/apt-get update && sleep 1', - refreshonly => true, - subscribe => [ File['/etc/apt/apt.conf.d'], Config_file['/etc/apt/sources.list'] ]; - } + include apt::dot_d_directories ## This package should really always be current package { "debian-archive-keyring": ensure => latest } |