- exec { "/usr/bin/apt-mark ${setting} ${title}":
- onlyif => "/usr/bin/dpkg -l ${title}",
- unless => $unless_cmd,
+
+ if $setting == 'unhold' {
+ $unless_cmd = undef
+ } else {
+ $action = "show${setting}"
+
+ # It would be ideal if we could break out this command in to an array of args, similar
+ # to $onlyif_cmd and $command. However, in this case it wouldn't work as expected due
+ # to the inclusion of a pipe character.
+ # When passed to the exec function, the posix provider will strip everything to the right of the pipe,
+ # causing the command to return a full list of packages for the given action.
+ # The trade off is to use an interpolated string knowing that action is built from an enum value and
+ # title is pre-validated.
+ $unless_cmd = ["/usr/bin/apt-mark ${action} ${title} | grep ${title} -q"]