geerlingguy / ansible-role-php-versions

Ansible Role - PHP Versions
https://galaxy.ansible.com/geerlingguy/php-versions/
MIT License
98 stars 73 forks source link

Add support for PHP 7.3 #27

Closed geerlingguy closed 5 years ago

geerlingguy commented 6 years ago

Should be pretty simple. Going to open a php-73 branch and see what fails currently.

geerlingguy commented 6 years ago

Ah, silly default package sets...

    TASK [geerlingguy.php : Ensure PHP packages are installed.] ********************
    fatal: [instance]: FAILED! => {"changed": false, "msg": "No package matching 'php7.3-mcrypt' is available"}

That's on Debian/Ubuntu builds with 7.3, since mcrypt has been deprecated (after being unmaintained for a decade or so). I'm thinking I can probably drop that dep in the default list at this point. I think most of the major CMSes don't have any particular mcrypt dependencies now. At least tested with Wordpress, Drupal 8, and Magento 2...