From 11ed2172dee62f0fa5c1c9d19eeccfba111a3003 Mon Sep 17 00:00:00 2001 From: Phil Porada Date: Mon, 2 May 2016 08:57:32 -0400 Subject: [PATCH] Per comments on PR106 --- README.md | 2 +- defaults/main.yml | 2 +- tasks/setup-Debian.yml | 2 +- tasks/setup-RedHat.yml | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/README.md b/README.md index 53c71b6..ee4f7a6 100644 --- a/README.md +++ b/README.md @@ -30,7 +30,7 @@ The default values for the HTTP server deamon are `httpd` (used by Apache) for R (RedHat/CentOS only) If you have enabled any additional repositories (might I suggest [geerlingguy.repo-epel](https://github.com/geerlingguy/ansible-role-repo-epel) or [geerlingguy.repo-remi](https://github.com/geerlingguy/ansible-role-repo-remi)), those repositories can be listed under this variable (e.g. `remi-php56,epel`). This can be handy, as an example, if you want to install the latest version of PHP 5.6, which is in the Remi repository. - php_repostate: "installed" + php_packages_state: "installed" If you have enabled any additional repositories such as [geerlingguy.repo-epel](https://github.com/geerlingguy/ansible-role-repo-epel) or [geerlingguy.repo-remi](https://github.com/geerlingguy/ansible-role-repo-remi), you may want an easy way to swap PHP versions on the fly. By default, this is set to 'installed'. You can now override this variable to 'latest'. Combined with php_enablerepo, a user now doesn't need to manually uninstall the existing PHP packages before installing them from a different repository. diff --git a/defaults/main.yml b/defaults/main.yml index 00c60ed..236d85a 100644 --- a/defaults/main.yml +++ b/defaults/main.yml @@ -3,7 +3,7 @@ php_enablerepo: "" # State that the system package installer should use for PHP the enabled repo (e.g. "installed,latest") -php_repostate: installed +php_packages_state: installed # Set this to false if you're not using PHP with Apache/Nginx/etc. php_enable_webserver: true diff --git a/tasks/setup-Debian.yml b/tasks/setup-Debian.yml index 7cbcc04..65d9629 100644 --- a/tasks/setup-Debian.yml +++ b/tasks/setup-Debian.yml @@ -5,7 +5,7 @@ - name: Ensure PHP packages are installed. apt: name: "{{ item }}" - state: "{{ php_repostate }}" + state: "{{ php_packages_state }}" with_items: "{{ php_packages }}" register: php_package_install notify: restart webserver diff --git a/tasks/setup-RedHat.yml b/tasks/setup-RedHat.yml index 15dd8fd..6cfde8c 100644 --- a/tasks/setup-RedHat.yml +++ b/tasks/setup-RedHat.yml @@ -2,7 +2,7 @@ - name: Ensure PHP packages are installed. yum: name: "{{ item }}" - state: "{{ php_repostate }}" + state: "{{ php_packages_state }}" enablerepo: "{{ php_enablerepo }}" with_items: "{{ php_packages }}" notify: restart webserver