geerlingguy / ansible-role-ntp

Ansible Role - NTP
https://galaxy.ansible.com/geerlingguy/ntp/
MIT License
319 stars 243 forks source link

FreeBSD comes with tzata in the base install, there is no package tzdata #39

Closed pdreker closed 7 years ago

pdreker commented 7 years ago

Though FreeBSD is not officially supported this role works quite fine with FreeBSD11. Unfortunately the fix for issue #33 changed that, as the role now errors out on FreeBSD, as there is no tzdata package.

This simply makes the installation of tzdata dependent on the target being 'ansible_system == "Linux"', which makes it work again.

geerlingguy commented 7 years ago

Good catch, sorry about that! I'll tag a new release in a few minutes.