GNU bug report logs - #25967
25.1; Support for ImageMagick 7

Previous Next

Package: emacs;

Reported by: Tej Chajed <tchajed <at> mit.edu>

Date: Sat, 4 Mar 2017 16:21:02 UTC

Severity: wishlist

Tags: patch

Found in version 25.1

Fixed in version 27.1

Done: Glenn Morris <rgm <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Karl Otness <karl <at> karlotness.com>
To: 25967 <at> debbugs.gnu.org
Subject: bug#25967: [PATCH] Add support for ImageMagick 7 (Bug#25967)
Date: Fri, 1 Jun 2018 14:17:32 -0500
[Message part 1 (text/plain, inline)]
I have attached a patch which adds support for ImageMagick version 7.
I have tried to keep compatibility with version 6 as well. I have
gotten clean builds against both versions on my machine (running Arch
and overriding PKG_CONFIG_PATH to test). I also did a quick build on a
Mac and it also seemed to link up to the ImageMagick 7 from homebrew.
On both machines I could open/resize/rotate images and page through
the frames of animated GIFs.

The changes that seem to be needed are doable with a #define and a
typedef to rename a function and type in image.c. The patch makes
these conditional on the major version that is found by pkg-config
during ./configure, trying version 7 first and falling back to 6.

I'm not very familiar with autoconf, but the conditional checking in
the patch seems to work and avoids checking for ImageMagick 6 if it
finds version 7 first. I used AS_IF for this rather than the action
arguments to EMACS_CHECK_MODULES since this way seems to avoid the
redundant checks.

Thanks,
Karl
[0001-Add-support-for-ImageMagick-7-Bug-25967.patch (text/x-patch, attachment)]

This bug report was last modified 6 years and 263 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.