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
Fixed circular dependency for package dirmngr
[puppet-modules/puppetlabs-apt.git]
/
manifests
/
setting.pp
diff --git
a/manifests/setting.pp
b/manifests/setting.pp
index f6f0bc0c2275821a3656d9e14cb73b30f1202f88..507ec9495d66ac2e6d305f71596996d34e7e1973 100644
(file)
--- a/
manifests/setting.pp
+++ b/
manifests/setting.pp
@@
-1,13
+1,11
@@
define apt::setting (
define apt::setting (
-
$priority
= 50,
-
$ensure
= file,
-
$source
= undef,
-
$content
= undef,
-
$file_perms = {}
,
+
Variant[String, Integer, Array] $priority
= 50,
+
Optional[Enum['file', 'present', 'absent']] $ensure
= file,
+
Optional[String] $source
= undef,
+
Optional[String] $content
= undef,
+
Boolean $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')
}
@@
-16,8
+14,6
@@
define apt::setting (
fail('apt::setting needs either of content or source')
}
fail('apt::setting needs either of content or source')
}
- validate_re($ensure, ['file', 'present', 'absent'])
-
$title_array = split($title, '-')
$setting_type = $title_array[0]
$base_name = join(delete_at($title_array, 0), '-')
$title_array = split($title, '-')
$setting_type = $title_array[0]
$base_name = join(delete_at($title_array, 0), '-')
@@
-29,15
+25,7
@@
define apt::setting (
validate_re($priority, '^\d+$', 'apt::setting priority must be an integer or a zero-padded integer')
}
validate_re($priority, '^\d+$', 'apt::setting priority must be an integer or a zero-padded integer')
}
- if $source {
- validate_string($source)
- }
-
- if $content {
- validate_string($content)
- }
-
- if $setting_type == 'list' {
+ if ($setting_type == 'list') or ($setting_type == 'pref') {
$_priority = ''
} else {
$_priority = $priority
$_priority = ''
} else {
$_priority = $priority
@@
-46,12
+34,19
@@
define apt::setting (
$_path = $::apt::config_files[$setting_type]['path']
$_ext = $::apt::config_files[$setting_type]['ext']
$_path = $::apt::config_files[$setting_type]['path']
$_ext = $::apt::config_files[$setting_type]['ext']
+ if $notify_update {
+ $_notify = Class['apt::update']
+ } else {
+ $_notify = undef
+ }
+
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,
content => $content,
source => $source,
+ notify => $_notify,
}
}
}
}