search

tanmng / docker-chevereto-free

Dockerfiles for various release of Chevereto Free
38 stars 26 forks source link

Add exif extension #2

Closed samsonnguyen closed 5 years ago

samsonnguyen commented 5 years ago

Chevereto uses the exif extension if available when uploading images to extract exif data. This pr simple enables the exif php extension.

With exif ext enabled:

root@chevereto-7fc7d8dc49-tfnws:/var/www/html# php -a
Interactive shell

php > print function_exists('read_exif_data') ? 'true' : 'false';
true
tanmng commented 5 years ago

Cool, Thank you very much for the PR @samsonnguyen .

Can you please re-arrange the parameters of the command docker-php-ext-install so that the list of extensions is in alphabetical order (it's a best practice recommended by Docker lab)

Also, please update the file Dockerfile-installer so that the exif extension is installed in the installer image as well.

tanmng commented 5 years ago

LGTM