Code Review
/
puppet-modules
/
puppetlabs-apt.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Merge pull request #496 from puppetlabs/allow-undef-content-conf
[puppet-modules/puppetlabs-apt.git]
/
manifests
/
setting.pp
diff --git
a/manifests/setting.pp
b/manifests/setting.pp
index c288b35d7ec00ce545af777de09bfe977026738c..ab844602326bc166d1cc6bf934db19905cbea098 100644
(file)
--- a/
manifests/setting.pp
+++ b/
manifests/setting.pp
@@
-3,12
+3,9
@@
define apt::setting (
$ensure = file,
$source = undef,
$content = undef,
$ensure = file,
$source = undef,
$content = undef,
- $file_perms = {},
$notify_update = true,
) {
$notify_update = true,
) {
- $_file = merge($::apt::file_defaults, $file_perms)
-
if $content and $source {
fail('apt::setting cannot have both content and source')
}
if $content and $source {
fail('apt::setting cannot have both content and source')
}
@@
-56,17
+53,11
@@
define apt::setting (
file { "${_path}/${_priority}${base_name}${_ext}":
ensure => $ensure,
file { "${_path}/${_priority}${base_name}${_ext}":
ensure => $ensure,
- owner =>
$_file['owner']
,
- group =>
$_file['group']
,
- mode =>
$_file['mode']
,
+ owner =>
'root'
,
+ group =>
'root'
,
+ mode =>
'0644'
,
content => $content,
source => $source,
notify => $_notify,
}
content => $content,
source => $source,
notify => $_notify,
}
-
- if $notify_update {
- anchor { "apt::setting::${name}":
- require => Class['apt::update']
- }
- }
}
}