Debian 9 also uses php-pear as CentOS does

geerlingguy / ansible-role-php-pecl

Ansible Role - PHP PECL extension

MIT License

32 stars 37 forks source link

Closed ajardan closed 5 years ago

ajardan commented 5 years ago

Added a new var file and changed the way var files are imported to include the OS major version.

geerlingguy commented 5 years ago

@ajardan - Thanks! To finish this out (and make sure we don't introduce any bugs to Debian 9 users again...), can you add a debian9 entry to the test suite? https://github.com/geerlingguy/ansible-role-php-pecl/blob/master/.travis.yml#L8-L10

Right now the role doesn't do a Debian 9 test, but it would be good to do so.

ajardan commented 5 years ago

Now it seems like php_default_version_debian does not work well on Debian 9. I see a few solutions here:

Use the versions role to set up stuff properly on Debian 9 (which will add necessary repos for 7.2)
Use a separate playbook for Ubuntu 18.04 since that is the only one requiring by default 7.2
The best option I have no idea about but you will suggest :)?

geerlingguy commented 5 years ago

Thanks!

ajardan commented 5 years ago

Thank you for merging this !