search

w0rd-driven / docker-laravel-deployer

Docker image build for use with Laravel 5.x, including deployer
MIT License
2 stars 2 forks source link

Add zip package to correct warnings in CI builds #11

Closed w0rd-driven closed 4 years ago

w0rd-driven commented 4 years ago 
[10.1MB/1.22s] As there is no 'unzip' command installed zip files are being unpacked using the PHP zip extension.
[10.1MB/1.22s] This may cause invalid reports of corrupted archives. Besides, any UNIX permissions (e.g. executable) defined in the archives will be lost.
[10.1MB/1.22s] Installing 'unzip' may remediate them.

This likely won't be a problem for PHP 7.3 as I'm adding that in but this does likely affect the 5.6, 7.0, 7.1, and 7.2 builds.

w0rd-driven commented 4 years ago

Fixed in PR #13