Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

php-pecl-imagick version 0:3.4.2-1.el7.remi.5.6 not work with ImageMagick-last-libs version 6.9.4.1-1.el7.remi #33

Closed
makhomed opened this issue May 13, 2016 · 3 comments

Comments

@makhomed
Copy link

php-pecl-imagick version 0:3.4.2-1.el7.remi.5.6
not work with ImageMagick-last-libs version 6.9.4.1-1.el7.remi
when runned via php-fpm version php-fpm-5.6.21-1.el7.remi:

<?php 
try {
        (new Imagick)->readImage('/home/www/example.com/static/content/files/image.jpg');
    } catch (Exception $e) {
        echo $e->getMessage();
    }
  die;

Exception:

no decode delegate for this image format `JPEG' @ error/constitute.c/ReadImage/501

workaround:
downgrade ImageMagick-last-libs versions to 6.9.3.10-1.el6.remi:

http://fr2.rpmfind.net/linux/remi/enterprise/6.7/safe/i386/ImageMagick-last-libs-6.9.3.10-1.el6.remi.i686.rpm

http://fr2.rpmfind.net/linux/remi/enterprise/7/remi/x86_64/ImageMagick-last-libs-6.9.3.10-1.el7.remi.x86_64.rpm

can you please fix this bug with php-pecl-imagick and latest version of ImageMagick-last-libs?

@remicollet
Copy link
Owner

Hmm... I cannot reproduce

# rpm -q ImageMagick-last-libs php-common php-pecl-imagick
ImageMagick-last-libs-6.9.4.1-1.el7.remi.x86_64
php-common-5.6.21-1.el7.remi.x86_64
php-pecl-imagick-3.4.2-1.el7.remi.5.6.x86_64

With you example (changed to display image size)

# php /tmp/foo.php 
Image is 86 x 133

And from phpinfo (jpeg is there):

imagick module => enabled
imagick module version => 3.4.2
imagick classes => Imagick, ImagickDraw, ImagickPixel, ImagickPixelIterator, ImagickKernel
Imagick compiled with ImageMagick version => ImageMagick 6.9.3-8 Q16 x86_64 2016-04-12 http://www.imagemagick.org
Imagick using ImageMagick library version => ImageMagick 6.9.4-1 Q16 x86_64 2016-05-11 http://www.imagemagick.org
ImageMagick copyright => Copyright (C) 1999-2016 ImageMagick Studio LLC
ImageMagick release date => 2016-05-11
ImageMagick number of supported formats:  => 225
ImageMagick supported formats => 3FR, AAI, AI, ART, ARW, AVI, AVS, BGR, BGRA, BGRO, BIE, BMP, BMP2, BMP3, BRF, CAL, CALS, CANVAS, CAPTION, CIN, CIP, CLIP, CMYK, CMYKA, CR2, CRW, CUR, CUT, DATA, DCM, DCR, DCX, DDS, DFONT, DNG, DOT, DPX, DXT1, DXT5, EPDF, EPI, EPS, EPS2, EPS3, EPSF, EPSI, EPT, EPT2, EPT3, ERF, EXR, FAX, FITS, FRACTAL, FTS, G3, GIF, GIF87, GRADIENT, GRAY, GROUP4, GV, H, HALD, HDR, HISTOGRAM, HRZ, HTM, HTML, ICB, ICO, ICON, IIQ, INFO, INLINE, IPL, ISOBRL, ISOBRL6, JBG, JBIG, JNG, JNX, JPE, JPEG, JPG, JPS, JSON, K25, KDC, LABEL, M2V, M4V, MAC, MAGICK, MAP, MASK, MAT, MATTE, MEF, MIFF, MKV, MNG, MONO, MOV, MP4, MPC, MPEG, MPG, MRW, MSL, MSVG, MTV, MVG, NEF, NRW, NULL, ORF, OTB, OTF, PAL, PALM, PAM, PANGO, PATTERN, PBM, PCD, PCDS, PCL, PCT, PCX, PDB, PDF, PDFA, PEF, PES, PFA, PFB, PFM, PGM, PICON, PICT, PIX, PJPEG, PLASMA, PNG, PNG00, PNG24, PNG32, PNG48, PNG64, PNG8, PNM, PPM, PREVIEW, PS, PS2, PS3, PSB, PSD, PTIF, PWP, RADIAL-GRADIENT, RAF, RAS, RAW, RGB, RGBA, RGBO, RGF, RLA, RLE, RMF, RW2, SCR, SCT, SFW, SGI, SHTML, SIX, SIXEL, SPARSE-COLOR, SR2, SRF, STEGANO, SUN, SVG, SVGZ, TEXT, TGA, THUMBNAIL, TIFF, TIFF64, TILE, TIM, TTC, TTF, TXT, UBRL, UBRL6, UIL, UYVY, VDA, VICAR, VID, VIFF, VIPS, VST, WBMP, WEBP, WMF, WMV, WMZ, WPG, X, X3F, XBM, XC, XCF, XPM, XPS, XV, XWD, YCbCr, YCbCrA, YUV

Are all images affected ?
Can you give an link to an example ?

Digging...

@makhomed
Copy link
Author

Sorry, this is my bug - I update ImageMagick-last-libs, but not restart php-fpm service, and got error

no decode delegate for this image format JPEG' @ error/constitute.c/ReadImage/501`

for all websites on server, running via php-fpm.

I was confused by this useless error message, sorry.

@remicollet
Copy link
Owner

Thanks for confirming there is no bug
(BTW, restart is needed, as the path to the "coders" change on each new version)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants