Skip to content

Commit b0111c2

Browse files
committed
Replace merge function with +
Latest stdlib (from git) has a new `merge` implementation. puppetlabs/puppetlabs-stdlib#1008 It appears to be broken for puppet 4. We're planning to drop puppet 4 any moment, but for right now, replacing the call to `merge` with `+` works fine. Fixes ``` Puppet::PreformattedError: Evaluation Error: Error while evaluating a Function Call, 'merge' expects one of: (Variant[Hash, Undef, String[0, 0]] args*) rejected: parameter 'args' variant 0 entry 'notify' expects a value of type Undef or Data, got Type[Class] rejected: parameter 'args' variant 1 expects a value of type Undef or String, got Struct (Iterable args*, Callable[3, 3] block) rejected: expects a block (Iterable args*, Callable[2, 2] block) rejected: expects a block at /home/travis/build/voxpupuli/puppet-jenkins/spec/fixtures/modules/jenkins/manifests/config.pp:45:16 ```
1 parent 5d292f1 commit b0111c2

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

manifests/config.pp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,12 @@
4242
}
4343

4444
$plugin_dir_params = $::jenkins::purge_plugins ? {
45-
true => merge($dir_params, {
45+
true => $dir_params + {
4646
'purge' => true,
4747
'recurse' => true,
4848
'force' => true,
49-
'notify' => Class['::jenkins::service'],
50-
}),
49+
'notify' => Class['jenkins::service'],
50+
},
5151
default => $dir_params,
5252
}
5353

0 commit comments

Comments
 (0)