search

geerlingguy / ansible-role-php

Ansible Role - PHP
https://galaxy.ansible.com/geerlingguy/php/
MIT License
496 stars 442 forks source link

feat: allow the configuration of any opcache variable #418

Open jpiron opened 5 months ago

jpiron commented 5 months ago

This removes the need to update the role to support new opcache variables.

shakalandy commented 5 months ago

Cool, this would make my PR #416 obsolete, but keep the default, right?

jpiron commented 5 months ago

Yes, except for the php_realpath_cache_ttl variable. \ And indeed it keeps all the defaults as long as you don't define the php_opcache_* variables.\ I updated the README to make it more clear.

shakalandy commented 5 months ago

@geerlingguy could we get this one merged? I would adapt #416 to include php_realpath_cache_ttl only - thanks!

github-actions[bot] commented 1 month ago

This pr has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.