search

geerlingguy / ansible-role-php

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

If apache2 is not used dont copy document on this path variables __php_conf_paths __php_extension_conf_paths #364

Closed yodatak closed 1 year ago

yodatak commented 2 years ago

When we don't used apache2 we don't need config to be droped in apache2 folder so we must fix this in the vars/Debian.yml If we don't need apache it must be like this

__php_conf_paths:

__php_extension_conf_paths:

__php_extension_conf_paths:

We must add a check in jinja2 that php_webserver_daemon is different from apache

stale[bot] commented 2 years ago

This issue 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.

yodatak commented 2 years ago

Up

stale[bot] commented 2 years ago

This issue is no longer marked for closure.

stale[bot] commented 1 year ago

This issue 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.

stale[bot] commented 1 year ago

This issue has been closed due to inactivity. If you feel this is in error, please reopen the issue or file a new issue with the relevant details.

yodatak commented 1 year ago

I reopen this are you open to a Pr @geerlingguy ?