Make WordPress Core

Opened 14 years ago

Closed 13 years ago

#23331 closed enhancement (maybelater)

Gmagick support

Reported by: markoheijnen Owned by:
Priority: normal Milestone:
Component: Media Version:
Severity: normal Keywords: has-patch 3.7-early
Cc: Focuses:

Description

In 3.5 we added Imagick support. It would be great to also include Gmagick. Since Graphicmagick is just more awesome then ImageMagick.

Attachments (3)

23331.diff​ (12.3 KB ) - added by markoheijnen 14 years ago.
First try. Code from ​http://wordpress.org/extend/plugins/gmagick/
23331.2.diff​ (12.3 KB ) - added by markoheijnen 14 years ago.
Change version number to 3.6.0 and also add [23750] to this patch
23331.3.diff​ (12.3 KB ) - added by dh-shredder 14 years ago.
Same patch, but with Gmagick last in line.

Download all attachments as: .zip

Change History (13)

#1 @markoheijnen
14 years ago

  • Keywords has-patch added

#2 @nacin
14 years ago

Sweet.

#3 @dh-shredder
14 years ago

If we're going to go for Gmagick support in 3.6, I'd suggest dropping it in for a soak as soon as we can, so that we can discover any hosting compatibility issues that might arise.

My only hesitation would be that there's less hosting support for Gmagick than Imagick. That said, it's pretty clear that Gmagick/Graphicsmagick is the way forward, both due to its developers support, and performance concerns.

#4 @markoheijnen
14 years ago

  • Keywords commit added

@markoheijnen
14 years ago

Change version number to 3.6.0 and also add [23750] to this patch

#5 @markoheijnen
14 years ago

Can we commit this? The code is pretty robust because it's almost the same at imagick. We do need user tests. I'm already running this code before 3.5 was even launched and never had issues.

@dh-shredder
14 years ago

Same patch, but with Gmagick last in line.

#6 @dh-shredder
14 years ago

As requested in dev chat, same patch, but with Gmagick last in line: 23331.3.diff​

#7 @nacin
13 years ago

  • Keywords 3.7-early added; commit removed
  • Milestone 3.6 → Future Release

We load all core editors, even if we chose an earlier one first. So let's do this in 3.7, possibly reworking this to only include gmagick when it makes sense. Unfortunately we can't stop including GD and IM as classes may already be extending them.

#8 @markoheijnen
13 years ago

Don't see why it needs to get another flow. Loading the editors isn't something really heavy.

#9 @wonderboymusic
13 years ago

  • Milestone Future Release → 3.7

these are all marked 3.7-early

#10 @nacin
13 years ago

  • Milestone 3.7
  • Resolution → maybelater
  • Status new → closed

I've chatted with markoheijnen about this. According to some research, Gmagick has an install base of roughly 0% of WordPress sites. (The same sampling showed that GD was available on 95-98%+ of installs, Imagick on 25%.) So for the time being, this belongs in a plugin.

Note: See TracTickets for help on using tickets.