Closed (fixed)
Project:
ImageMagick
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
19 Feb 2011 at 13:05 UTC
Updated:
26 Apr 2013 at 14:43 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
parasox commentedWow yeah that looks nice.. See: http://www.graphicsmagick.org/benchmarks.html
Also used by Flickr and Etsy.. Would be nice to add Drupal to that list. :)
Comment #2
W32 commentedI made a port of this module to work with GraphicsMagick. On my local machine works fine.
You can find this port in my sanbox - http://drupal.org/sandbox/W32/1077458.
Comment #3
sun@W32: Can you provide a patch against ImageMagick module?
Comment #4
W32 commentedNo, GraphicsMagick is separate module. But you can install both - this and GraphicsMagick modules and switch from them in drupal admin/config/media/image-toolkit page.
Comment #5
sunDid a diff between imagemagick and that sandbox - http://drupal.org/sandbox/W32/1077458 is a 99.99% copy of http://drupal.org/project/imagemagick, the only changes are:
Kinda expected result, since GraphicsMagick itself is a rather stupid fork of ImageMagick.
Hence, looks like we support GraphicsMagick natively. Will clarify on the project page.
Comment #7
osopolarThe GraphicsMagick support is almost natively, but there are some differences. The syntax is: gm convert [options] input_file output_file.
I added a menu option to enable GraphicsMagick and made the necessary changes. In some places the term ImageMagick is still used while it should be GraphicsMagick.
I tested only to resize images (with GraphicsMagick 1.1.11 on Debian 4.3.2).
Patch attached.
Comment #8
sunThanks. Slightly cleaner variant of that attached.
However, I'm not able to test this myself, as I don't have gm installed anywhere (and don't intend to do so currently).
The run-time logic looks fine to me, but I'd appreciate excessive testing of the image toolkit settings form; i.e., checking gm version, switching toolkits back and forth, absolute path to gm binary, just the gm binary as path, etc.
Comment #9
osopolarI successfully tested the patch in #8 but only to resize images (with GraphicsMagick 1.1.11 on Debian 4.3.2). Maybe later some more testing.
Comment #10
fuerst commentedInstead of having the user choose between ImageMagick or GraphicsMagick you may auto-select it by looking at the path to the toolkit or by scanning the output of the convert -version command. See also here: http://drupal.org/node/1227636
Comment #11
sunThanks for reporting, reviewing, and testing! Committed #8 to 7.x-1.x.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.
This means that GraphicsMagick can actually be used.
We can further investigate a more automated detection of ImageMagick/GraphicsMagick prior to rolling a stable release. If you'd like to work on something along the proposal in #10, please create a new issue and submit a patch.
Comment #13
ericmulder1980 commentedSeems that assumption is the mother of all ... (fill in blanks). GraphicsMagick is NOT a 99.9% clone of ImageMagick. It was forked from ImageMagick back in 2001 and had it's first release in 2003. So every change between 2003 will not have 100% compatibility with this module.
I have an imagestyle with scale and define canvas options , this simply fails to render an image and gives me the following error:
User error: ImageMagick error 1: /usr/bin/gm convert: Unrecognized compose operator (src-over). in _imagemagick_convert_exec() (regel 519 van /var/www/
/htdocs/sites/all/modules/contrib/imagemagick/imagemagick.module).
I found this (imageapi_graphicsmagick) module and it basicly tells you the same thing. Too bad this module is only for D6 though.
So assuming this module works with GraphicsMagick out of the box is wrong. I love the ImageMagick module but the message on the module page will get a lot of people in trouble.
Comment #14
jacobpov commentedsame issue here User error: ImageMagick error 1: /usr/bin/gm convert: Unrecognized compose operator (src-over). in _imagemagick_convert_exec() (line 519 of /var/www/sites/all/modules/imagemagick/imagemagick.module).
Comment #15
rogerrogers commentedHave same problem as #13, bummah.
Comment #16
Triumphent commentedAll of this is very confusing. I am able to convert and scale animated GIFs (most of them) but it takes between 10 and 15 minutes per GIF file. In the mean time, the site freezes! :(