search

puppetlabs-toy-chest / puppetlabs-package_updates

Package update discovery Puppet Face and custom Facter fact
12 stars 6 forks source link

weekday => 'all' fails on solaris #18

Open purplexa opened 8 years ago

purplexa commented 8 years ago

Solaris fails with weekday => 'all' because 7 is not a valid weekday number per the POSIX definition of the crontab format (Vixie cron supports 0 or 7 to refer to Sunday, but the POSIX formats only describes 0, and Solaris tends to be stricter about these things).

Either weekday => 'all' should mean range(0,6) or, more simply, just set the value to '*' or 'absent', either of which will accomplish the same thing.

gordonmessmer commented 6 years ago

Does PR #29 fix this for you?