mingw issues - INT32 redefinition

GoogleCodeExporter commented 8 years ago

Attempted to build libwebp version 0.1.3 on mingw with msys.  When it tried to 
compile cwebp.c, it had a problem with redefinition of INT32.  Checking the 
jmorecfg.h header, it has ifdefs for redefinition:
  #ifndef _BASETSD_H        /* MinGW is slightly different */ 
However, one has to include the Windows includes (that pull in basetsd.h) first 
for the ifdef to come into play.

I moved the jpeg include files below the windows include files in cwebp.c and 
everything seems to build with no problems.

Am attaching the patch I created and used.

Original issue reported on code.google.com by lme...@gmail.com on 4 May 2012 at 4:52

Merged into: #102

Attachments:

patch.diff

GoogleCodeExporter commented 8 years ago

Thanks, this has already been fixed I believe and will be included in the next 
release:

commit 2e74ec8b5fae9799610266b6f0ee77b7a4dcbf20
Author: Pascal Massimino <pascal.massimino@gmail.com>
Date:   Thu Jan 12 04:51:39 2012 -0800

    fix compile under MINGW

    there was a type conflict around INT32 (in libjpeg).

    Patch by nipmarsh
    Issue 102: http://code.google.com/p/webp/issues/detail?id=102

    Change-Id: I6425b6b9b9861bcaa8bb1adc8a78541e44031a6e

Original comment by jz...@google.com on 4 May 2012 at 5:28

Changed state: Duplicate

GoogleCodeExporter commented 8 years ago

Thanks.  Ran a search for mingw related issues before posting, but it didn't 
come up (probably because the other issue was already closed and search default 
is for open).

Original comment by lme...@gmail.com on 4 May 2012 at 5:33

jaewonjoo / webp

mingw issues - INT32 redefinition #119