Skip to content

Conversation

@rhea-machine
Copy link
Contributor

The php.ini template was only used if php_version < 7.0 or os != debian.
That doesn't make sense as a few variables having effect on it don't relate to any of those conditions. Also, I need some of those (like php_upload_max_filesize) in our Moodle projects (using Debian and PHP >= 7.0).
Therefore I propose to include it by default in the PHP role.

  • This PR is an Improvement

@rhea-machine rhea-machine force-pushed the php-ini-anywhere branch 2 times, most recently from e9b9988 to b491339 Compare August 13, 2019 16:15
Copy link

@sephii sephii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me except for the small change to be done. @chregu was there a reason for the custom php.ini file to be used only on non-debian or PHP < 7 boxes?

# notify: restart apache
# become: yes
- name: Configure mod-php
command: "{{ phpenmod }} {{ item }}"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
command: "{{ phpenmod }} {{ item }}"
command: "{{ phpenmod }} php-dev"

And remove the with_items below.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants