PHP 7.2 removes mcrypt - Githubissues

w0rd-driven / docker-laravel-deployer

Docker image build for use with Laravel 5.x, including deployer

MIT License

2 stars 2 forks source link

PHP 7.2 removes mcrypt #2

Closed w0rd-driven closed 6 years ago

w0rd-driven commented 6 years ago

Remove mcrypt to get the 7.2 and latest tags building again. It's unclear if dompdf needs this again but it doesn't look like it. I'm unable to track when the requirement changed and it may only be a suggestion according to other composer.lock files I've seen.

If this breaks CI builds we'll know to add it back now that we know how.

w0rd-driven commented 6 years ago

I'll assume I fixed this one as well. I should've used pull requests.