Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#23331 closed enhancement (maybelater)

Gmagick support

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

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 11 years ago.
First try. Code from http://wordpress.org/extend/plugins/gmagick/
23331.2.diff (12.3 KB) - added by markoheijnen 11 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 11 years ago.
Same patch, but with Gmagick last in line.

Download all attachments as: .zip

Change History (13)

#1 @markoheijnen
11 years ago

  • Keywords has-patch added

#2 @nacin
11 years ago

Sweet.

#3 @dh-shredder
11 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
11 years ago

  • Keywords commit added

@markoheijnen
11 years ago

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

#5 @markoheijnen
11 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
11 years ago

Same patch, but with Gmagick last in line.

#6 @dh-shredder
11 years ago

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

#7 @nacin
11 years ago

  • Keywords 3.7-early added; commit removed
  • Milestone changed from 3.6 to 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
11 years ago

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

#9 @wonderboymusic
11 years ago

  • Milestone changed from Future Release to 3.7

these are all marked 3.7-early

#10 @nacin
11 years ago

  • Milestone 3.7 deleted
  • Resolution set to maybelater
  • Status changed from new to 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.