unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: "Jan Djärv" <jan.h.d@swipnet.se>
To: Tassilo Horn <tassilo@member.fsf.org>
Cc: Kalle Kankare <kalle.kankare@iki.fi>, 11558-done@debbugs.gnu.org
Subject: bug#11558: 24.1.50; Image Magick related build failure on bzr revision 108365
Date: Fri, 13 Jul 2012 14:21:42 +0200	[thread overview]
Message-ID: <7C929D90-A0E8-4DA3-97E1-152D8662CC57@swipnet.se> (raw)
In-Reply-To: <87629sdty3.fsf@thinkpad.tsdh.de>

Hello.

I installed the patch, lets see if it fails on some version :-).

Thanks,

	Jan D.

13 jul 2012 kl. 09:03 skrev Tassilo Horn:

> Jan Djärv <jan.h.d@swipnet.se> writes:
> 
>>> I got the build to complete by introducing the following change:
>>> 
>>> --- src/image.c	2012-07-09 12:02:27 +0000
>>> +++ src/image.c	2012-07-11 19:59:24 +0000
>>> @@ -7968,7 +7968,9 @@
>>>  Lisp_Object typelist = Qnil;
>>>  size_t numf = 0;
>>>  ExceptionInfo ex;
>>> +  GetExceptionInfo(&ex);
>>>  char **imtypes = GetMagickList ("*", &numf, &ex);
>>> +  DestroyExceptionInfo(&ex);
>>>  size_t i;
>>>  Lisp_Object Qimagemagicktype;
>>>  for (i = 0; i < numf; i++)
>>> 
>>> 
>>> The generated emacs-binary at least appears to function properly. I
>>> tested this with the following versions:
>>> 
>>> emacs: bzr:109033
>>> gcc: 4.7.1
>>> ImageMagick: 6.7.8-1
>> 
>> I see this too.  Can it be that GetExceptionInfo/DestroyExceptionInfo
>> is version dependent and we need a configure check for them? People
>> have run Emacs with Imagemagic without this.
> 
> I build and run emacs with imagemagick support just fine.  The IM
> version is 6.7.7.5, so it might be a version difference between 6.7.7
> and 6.7.8.
> 
> I also tried building and running with the above patch, and it doesn't
> seem to cause any harm.
> 
> Bye,
> Tassilo






      reply	other threads:[~2012-07-13 12:21 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-25 15:35 bug#11558: 24.1.50; Image Magick related build failure on bzr revision 108365 Eric Schulte
2012-05-25 20:21 ` Glenn Morris
2012-05-26 22:26   ` Eric Schulte
2012-05-26 22:33     ` Glenn Morris
2012-05-26 23:31       ` Eric Schulte
2012-05-27  3:08         ` Christoph Scholtes
2012-05-27  5:58 ` Achim Gratz
2012-07-11 20:24 ` Kalle Kankare
2012-07-12  9:19   ` Jan Djärv
2012-07-13  7:03     ` Tassilo Horn
2012-07-13 12:21       ` Jan Djärv [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7C929D90-A0E8-4DA3-97E1-152D8662CC57@swipnet.se \
    --to=jan.h.d@swipnet.se \
    --cc=11558-done@debbugs.gnu.org \
    --cc=kalle.kankare@iki.fi \
    --cc=tassilo@member.fsf.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).