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

Previous Next

Full log

View this message in rfc822 format

From: Andy Moreton <andrewjmoreton <at> gmail.com>
To: 25967 <at> debbugs.gnu.org
Subject: bug#25967: [PATCH] Add support for ImageMagick 7 (Bug#25967)
Date: Tue, 04 Sep 2018 20:03:07 +0100

On Tue 04 Sep 2018, Eli Zaretskii wrote:

>> From: Andy Moreton <andrewjmoreton <at> gmail.com>
>> Date: Mon, 03 Sep 2018 23:57:05 +0100
>> 
>> As a followup, here is a patch to add ImageMagick support for Windows.
>> The patch was initially developed for Imagemagick 6, but I've updated it
>> to support ImageMagick 7.
>> 
>> I've given this some light testing on a 64bit mingw64 (MSYS2) and 64bit
>> cygwin builds, both of which use Imagemagick 7.
>> 
>> Please test, and report if it breaks anything on other platforms.
>
> Thanks.  A couple of minor comments:
>
> This needs a NEWS entry.
>
>> +       '(imagemagick "libMagickWand-7.Q16HDRI-6.dll")))
>
> Is this DLL name fixed for all the supported versions?  It sounds
> like it's only for Imagemagick v7, and so the DLL for version 6 will
> be named differently.

Indeed. v6 is legacy (and not available on any platform I currently test
on). The DLL name is from the current package for MSYS2.

> Also, AFAIU, there are binary incompatibilities between v6 and v7, so
> an Emacs compiled with one of them should not attempt to load DLLs
> from another, is that right?

Correct.

> For these two reasons, I think we should have a Lisp variable that
> provides the version of Imagemagick with which Emacs was built, and we
> need the dispatch in w32-win.el for loading the correct DLLs based on
> that variable, like we do with libgif etc.

Agreed. Do you use Imagemagick on mingw.org builds ? Is there a packaged
library available there ?

    AndyM

Previous Next

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