[Vote] Support JPEG images about laravel-docker HOT 7 CLOSED

Hi there 👋

Thanks for reporting this. Could you please fork this and try to add the libjpeg-dev library? If that fixes your issue, I'll add it here too.

Let me now. 🤞

from laravel-docker.

This helped me.

https://hub.docker.com/_/php/, under "PHP Core Extensions".

RUN apt-get install -y \
        libfreetype6-dev \
        libjpeg62-turbo-dev \
    && docker-php-ext-configure gd --with-freetype-dir=/usr/include/ --with-jpeg-dir=/usr/include/

This added JPEG support to my Dockerfile and made the imagejpeg() function missing error disappear.

from laravel-docker.

Thanks for sharing @anroots.

I'll try and implement this natively soon.

Feel free to open a PR if you want to speed up the process. 🙂

from laravel-docker.

Well I gave it a go https://github.com/richstandbrook/laravel-docker/blob/master/latest/Dockerfile#L22 but it still seems to have a problem with the same test. It can't be a fundamental problem with Laravel implementation as it works-for-me™ on my local php environment and I can't find reference to the problem anywhere else. It's a strange one.

from laravel-docker.

Strange, sorry I've never come across this issue before. May I ask why you're using libjpeg-turbo-dev instead of libjpeg-dev?

from laravel-docker.

Okay so I've had to manually add exif to the 7.4 image and in my research, I saw that they simplified the process of configuring gd with RUN docker-php-ext-configure gd --with-freetype --with-jpeg. So I went ahead and decided to support JPEG for 7.4 out of the box.

I'm a bit worried about the breaking changes of backporting this to 7.2 and 7.3 so I'm not sure if I should...

from laravel-docker.

Okay so for the sake of consistency and to make sure all images are supporting the same extensions and features, I've added support for JPEG on 7.2 and 7.3.

It's currently building on Docker Hub and should be available in a few minutes.

from laravel-docker.