daemon_options appears to be broken with the latest stdlib

[smoeding]

daemon_options appears to be broken with the latest stdlib

[jyaworski]

FYI, the code in question is here:

https://github.com/smoeding/puppet-sendmail/blob/master/manifests/mc/daemon_options.pp

[smoeding]

When you say you updated stdlib, you talk about stdlib version 4.11.0, right? Or are you running stdlib from the current git repository?

[smoeding]

I believe you introduced an incompatible change to the delete function.
delete(['ab','b','c','b'], 'b') used to return [ 'ab', 'c' ] before your recent change. I think now it would only return [ 'c' ].

[jyaworski]

Yep, thanks @smoeding. This isn't you.

puppetlabs/puppetlabs-stdlib#592

[jyaworski]

Reopening this because there was a stdlib update that touched this, and it's still broken.

-DAEMON_OPTIONS(`Name=MTA-v4, Family=inet, Port=smtp')dnl
-DAEMON_OPTIONS(`Name=MTA-v6, Family=inet6, Port=smtp, M=O')dnl
+DAEMON_OPTIONS(`=, =, =, =, =, =, =, =, =, =, =')dnl
+DAEMON_OPTIONS(`=, =, =, =, =, =, =, =, =, =')dnl

This is from using an old stdlib vs updating to a new one. Maybe @bmjen and @hunner will have an opinion.

[jyaworski]

It looks like even with that PR merged, I still get this problem. There haven't been any changes to the delete function.

[jyaworski]

PEBKAC.