geerlingguy / ansible-role-php-versions

Ansible Role - PHP Versions
https://galaxy.ansible.com/geerlingguy/php-versions/
MIT License
98 stars 73 forks source link

fix: debian vagrant box error for dirmngr install #87

Closed marsante closed 1 year ago

marsante commented 1 year ago

Thank for your great work.

I found a bug when I use the role with the debian vagrant box (bullseye and bookworm). Apt cache isn't update by defaultin the box and the role stop with this message during dirmngr install :

TASK [geerlingguy.php-versions : Ensure dirmngr is installed (gnupg dependency).] *** fatal: [medshakeehr]: FAILED! => {"changed": false, "msg": "No package matching 'dirmngr' is available"}

My fix correct this behavior.

geerlingguy commented 1 year ago

I typically add any apt cache management tasks on the playbook level; if I managed it in roles, I would have to have a task in every role to manage apt caches, and that's not as efficient.

marsante commented 1 year ago

Oh ok. I understand how you work better. Thanks