Skip to content

Commit 6c53cb7

Browse files
authored
Merge pull request #1113 from bastelfreak/pdksync_CONT-1028-update-stdlib-dependency
(CONT-1028) puppetlabs/stdlib: Allow 9.x
2 parents 91c8762 + c0b8640 commit 6c53cb7

File tree

5 files changed

+15
-19
lines changed

5 files changed

+15
-19
lines changed

manifests/init.pp

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@
200200
assert_type(Integer, $update['tries'])
201201
}
202202

203-
$_update = merge($apt::update_defaults, $update)
203+
$_update = $apt::update_defaults + $update
204204
include apt::update
205205

206206
if $purge['sources.list'] {
@@ -222,11 +222,11 @@
222222
assert_type(Boolean, $purge['apt.conf.d'])
223223
}
224224

225-
$_purge = merge($apt::purge_defaults, $purge)
225+
$_purge = $apt::purge_defaults + $purge
226226

227227
if $proxy['perhost'] {
228228
$_perhost = $proxy['perhost'].map |$item| {
229-
$_item = merge($apt::proxy_defaults, $item)
229+
$_item = $apt::proxy_defaults + $item
230230
$_scheme = $_item['https'] ? {
231231
true => 'https',
232232
default => 'http',
@@ -239,17 +239,13 @@
239239
true => 'DIRECT',
240240
default => "${_scheme}://${_item['host']}${_port}/",
241241
}
242-
merge($item, {
243-
'scheme' => $_scheme,
244-
'target' => $_target,
245-
}
246-
)
242+
$item + { 'scheme' => $_scheme, 'target' => $_target, }
247243
}
248244
} else {
249245
$_perhost = {}
250246
}
251247

252-
$_proxy = merge($apt::proxy_defaults, $proxy, { 'perhost' => $_perhost })
248+
$_proxy = $apt::proxy_defaults + $proxy + { 'perhost' => $_perhost }
253249

254250
$confheadertmp = epp('apt/_conf_header.epp')
255251
$proxytmp = epp('apt/proxy.epp', { 'proxies' => $_proxy })
@@ -386,12 +382,12 @@
386382
case $facts['os']['name'] {
387383
'Debian': {
388384
if versioncmp($facts['os']['release']['major'], '9') >= 0 {
389-
ensure_packages(['gnupg'])
385+
stdlib::ensure_packages(['gnupg'])
390386
}
391387
}
392388
'Ubuntu': {
393389
if versioncmp($facts['os']['release']['full'], '17.04') >= 0 {
394-
ensure_packages(['gnupg'])
390+
stdlib::ensure_packages(['gnupg'])
395391
}
396392
}
397393
default: {

manifests/key.pp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,13 +65,13 @@
6565
case $facts['os']['name'] {
6666
'Debian': {
6767
if versioncmp($facts['os']['release']['major'], '9') >= 0 {
68-
ensure_packages(['gnupg'])
68+
stdlib::ensure_packages(['gnupg'])
6969
Apt::Key<| title == $title |>
7070
}
7171
}
7272
'Ubuntu': {
7373
if versioncmp($facts['os']['release']['full'], '17.04') >= 0 {
74-
ensure_packages(['gnupg'])
74+
stdlib::ensure_packages(['gnupg'])
7575
Apt::Key<| title == $title |>
7676
}
7777
}

manifests/ppa.pp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@
7373

7474
if $ensure == 'present' {
7575
if $package_manage {
76-
ensure_packages($package_name)
76+
stdlib::ensure_packages($package_name)
7777
$_require = [File['sources.list.d'], Package[$package_name]]
7878
} else {
7979
$_require = File['sources.list.d']

manifests/source.pp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -107,14 +107,14 @@
107107
# Newer oses, do not need the package for HTTPS transport.
108108
$_transport_https_releases = ['9']
109109
if (fact('os.release.major') in $_transport_https_releases) and $_location =~ /(?i:^https:\/\/)/ {
110-
ensure_packages('apt-transport-https')
110+
stdlib::ensure_packages('apt-transport-https')
111111
Package['apt-transport-https'] -> Class['apt::update']
112112
}
113113
} else {
114114
$_location = undef
115115
}
116116

117-
$includes = merge($apt::include_defaults, $include)
117+
$includes = $apt::include_defaults + $include
118118

119119
if $key and $keyring {
120120
fail('parameters key and keyring are mutualy exclusive')
@@ -125,7 +125,7 @@
125125
unless $key['id'] {
126126
fail('key hash must contain at least an id entry')
127127
}
128-
$_key = merge($apt::source_key_defaults, $key)
128+
$_key = $apt::source_key_defaults + $key
129129
} else {
130130
$_key = { 'id' => assert_type(String[1], $key) }
131131
}
@@ -164,7 +164,7 @@
164164

165165
if $pin {
166166
if $pin =~ Hash {
167-
$_pin = merge($pin, { 'ensure' => $ensure, 'before' => $_before })
167+
$_pin = $pin + { 'ensure' => $ensure, 'before' => $_before }
168168
} elsif ($pin =~ Numeric or $pin =~ String) {
169169
$url_split = split($location, '[:\/]+')
170170
$host = $url_split[1]

metadata.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
"dependencies": [
1111
{
1212
"name": "puppetlabs/stdlib",
13-
"version_requirement": ">= 4.16.0 < 9.0.0"
13+
"version_requirement": ">= 4.16.0 < 10.0.0"
1414
}
1515
],
1616
"operatingsystem_support": [

0 commit comments

Comments
 (0)