all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Kazuhiro Ito <kzhr@d1.dion.ne.jp>
To: Eli Zaretskii <eliz@gnu.org>
Cc: 12464@debbugs.gnu.org
Subject: bug#12464: 24.2.50; nt/configure.bat fails to detect giflib-5.0.0
Date: Tue, 18 Sep 2012 19:32:21 +0900	[thread overview]
Message-ID: <87wqzraafe.wl%%xmue@d1.dion.ne.jp> (raw)
In-Reply-To: <83haqv1wbp.fsf@gnu.org>

> > Because gif_lib.h of giflib-5.0.0 assumes size_t is defined (I don't
> > know whether that is intended), nt/configure.bat fails to check its
> > availability.
> > 
> > In nt/config.log
> > > In file included from junk.c:1:0:
> > > f:/build/emacs-w32/include/gif_lib.h:262:37: error: expected ';', ',' or ')' before 'GifExtensionLength'
> > > f:/build/emacs-w32/include/gif_lib.h:265:8: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'EGifGCBToExtension'
> > > The failed program was: 
> > > #include "gif_lib.h" 
> > > main (){} 
> > > gcc -If:/build/emacs-w32/include -If:/build/emacs-w32/include/libxml2 -g  -c junk.c -o junk.obj 
> > > gcc -If:/build/emacs-w32/include -If:/build/emacs-w32/include/libxml2 -g  -c junk.c -o junk.obj 
> > 
> > Though I don't know whether Emacs supports giflib-5.0.0, the below
> > patch can fix the problem.  Gif support on my binaries (built by gcc
> > 4.5.2, MinGW) has no problem with giflib-5.0.0 as far as I tested.
> > 
> > === modified file 'nt/configure.bat'
> > --- nt/configure.bat	2012-09-08 11:20:32 +0000
> > +++ nt/configure.bat	2012-09-18 03:14:43 +0000
> > @@ -627,7 +627,8 @@
> >  if (%gifsupport%) == (N) goto gifDone
> >  
> >  echo Checking for libgif...
> > -echo #include "gif_lib.h" >junk.c
> > +echo #include "stdio.h" >junk.c
> > +echo #include "gif_lib.h" >>junk.c
> >  echo main (){} >>junk.c
> >  rem   -o option is ignored with cl, but allows result to be consistent.
> >  echo %COMPILER% %usercflags% %mingwflag% -c junk.c -o junk.obj >>config.log
> 
> Does it work to use stddef.h instead of stdio.h?  stddef.h is where
> size_t is normally defined.

Yes, stddef.h also works.  I didn't care where size_t is defined.

-- 
Kazuhiro Ito





  reply	other threads:[~2012-09-18 10:32 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-18  9:13 bug#12464: 24.2.50; nt/configure.bat fails to detect giflib-5.0.0 Kazuhiro Ito
2012-09-18 10:04 ` Eli Zaretskii
2012-09-18 10:32   ` Kazuhiro Ito [this message]
2012-09-18 10:52     ` Eli Zaretskii

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

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

  git send-email \
    --in-reply-to=87wqzraafe.wl%%xmue@d1.dion.ne.jp \
    --to=kzhr@d1.dion.ne.jp \
    --cc=12464@debbugs.gnu.org \
    --cc=eliz@gnu.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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.