[packages/exact-image] - added giflib patch (giflib 5+ support, covers both 5.0 and 5.1+)

qboosh qboosh at pld-linux.org
Mon Oct 20 17:53:15 CEST 2014


commit e293edfcce9d1c4d57d74fea75fe7e0c53aefeb1
Author: Jakub Bogusz <qboosh at pld-linux.org>
Date:   Mon Oct 20 17:53:59 2014 +0200

    - added giflib patch (giflib 5+ support, covers both 5.0 and 5.1+)

 exact-image-giflib.patch | 139 +++++++++++++++++++++++++++++++++++++++++++++++
 exact-image.spec         |   6 +-
 2 files changed, 143 insertions(+), 2 deletions(-)
---
diff --git a/exact-image.spec b/exact-image.spec
index 7a849e8..8b45dea 100644
--- a/exact-image.spec
+++ b/exact-image.spec
@@ -1,7 +1,7 @@
 #
 # Conditional build:
 %bcond_without	evas	# Edisplay support
-%bcond_with	gif	# GIF support
+%bcond_without	gif	# GIF support
 %bcond_without	lua	# Lua API
 %bcond_without	perl	# Perl API
 %bcond_with	php	# PHP API
@@ -23,6 +23,7 @@ Source0:	http://dl.exactcode.de/oss/exact-image/%{name}-%{version}.tar.bz2
 # Source0-md5:	a8694722cd7cc9aa9407950a8440f0cd
 Patch0:		%{name}-libs.patch
 Patch1:		exactimage_0.8.5-1.patch
+Patch2:		%{name}-giflib.patch
 URL:		http://www.exactcode.de/site/open_source/exactimage/
 BuildRequires:	OpenEXR-devel >= 1.2.0
 BuildRequires:	agg-devel >= 2.3
@@ -30,7 +31,7 @@ BuildRequires:	agg-devel >= 2.3
 BuildRequires:	expat-devel
 # pkgconfig(freetype) >= 9.5.0
 BuildRequires:	freetype-devel >= 2.1.6
-%{?with_gif:BuildRequires:	giflib4-devel}
+%{?with_gif:BuildRequires:	giflib-devel >= 5}
 BuildRequires:	jasper-devel
 BuildRequires:	lcms-devel >= 1.10
 BuildRequires:	libjpeg-devel
@@ -62,6 +63,7 @@ ImageMagick.
 %setup -q
 %patch0 -p1
 %patch1 -p1
+%patch2 -p1
 
 %build
 ./configure \
diff --git a/exact-image-giflib.patch b/exact-image-giflib.patch
new file mode 100644
index 0000000..f15c7ad
--- /dev/null
+++ b/exact-image-giflib.patch
@@ -0,0 +1,139 @@
+--- exact-image-0.8.9/codecs/gif.cc.orig	2010-03-03 22:04:44.000000000 +0100
++++ exact-image-0.8.9/codecs/gif.cc	2014-10-20 16:45:48.021255431 +0200
+@@ -17,6 +17,12 @@
+ 
+ #include <gif_lib.h>
+ 
++#if (GIFLIB_MAJOR > 5) || (GIFLIB_MINOR >= 1)
++#define Internal_EGifCloseFile(f) EGifCloseFile(f, NULL)
++#else
++#define Internal_EGifCloseFile(f) EGifCloseFile(f)
++#endif
++
+ #include "gif.hh"
+ #include "Colorspace.hh"
+ 
+@@ -58,11 +64,11 @@
+   GifRecordType RecordType;
+   GifByteType* Extension;
+   ColorMapObject *ColorMap = NULL;
+-  int ExtCode;
++  int ExtCode, GifError;
+   
+-  if ((GifFile = DGifOpen (stream, &GIFInputFunc)) == NULL)
++  if ((GifFile = DGifOpen (stream, &GIFInputFunc, &GifError)) == NULL)
+     {
+-      PrintGifError();
++      std::cerr << "Error: " << GifErrorString(GifError) << std::endl;
+       return false;
+     }
+   
+@@ -74,7 +80,7 @@
+   /* Scan the content of the GIF file and load the image(s) in: */
+   do {
+     if (DGifGetRecordType(GifFile, &RecordType) == GIF_ERROR) {
+-      PrintGifError();
++      std::cerr << "DGifGetRecordType error: " << GifErrorString(GifFile->Error) << std::endl;
+       return false;
+     }
+     
+@@ -83,7 +89,7 @@
+     switch (RecordType) {
+     case IMAGE_DESC_RECORD_TYPE:
+       if (DGifGetImageDesc(GifFile) == GIF_ERROR) {
+-	PrintGifError();
++        std::cerr << "DGifGetImageDesc error: " << GifErrorString(GifFile->Error) << std::endl;
+ 	return false;
+       }
+       
+@@ -104,7 +110,7 @@
+ 	       j += InterlacedJumps[i]) {
+ 	    if (DGifGetLine(GifFile, &image.getRawData()[j*image.stride()+Col],
+ 			    Width) == GIF_ERROR) {
+-	      PrintGifError();
++	      std::cerr << "DGifGetLine error: " << GifErrorString(GifFile->Error) << std::endl;
+ 	      return false;
+ 	    }
+ 	  }
+@@ -113,7 +119,7 @@
+ 	for (int i = 0; i < Height; ++i) {
+ 	  if (DGifGetLine(GifFile, &image.getRawData()[Row++ * image.stride()+Col],
+ 			  Width) == GIF_ERROR) {
+-	    PrintGifError();
++	    std::cerr << "DGifGetLine error: " << GifErrorString(GifFile->Error) << std::endl;
+ 	    return false;
+ 	  }
+ 	}
+@@ -122,12 +128,12 @@
+     case EXTENSION_RECORD_TYPE:
+       /* Skip any extension blocks in file: */
+       if (DGifGetExtension(GifFile, &ExtCode, &Extension) == GIF_ERROR) {
+-	PrintGifError();
++	std::cerr << "DGifGetExtension error: " << GifErrorString(GifFile->Error) << std::endl;
+ 	return false;
+       }
+       while (Extension != NULL) {
+ 	if (DGifGetExtensionNext(GifFile, &Extension) == GIF_ERROR) {
+-	  PrintGifError();
++	  std::cerr << "DGifGetExtensionNext error: " << GifErrorString(GifFile->Error) << std::endl;
+ 	  return false;
+ 	}
+       }
+@@ -155,7 +161,7 @@
+   // convert colormap to our 16bit "TIFF"format
+   colorspace_de_palette (image, ColorMap->ColorCount, rmap, gmap, bmap);
+   
+-  EGifCloseFile(GifFile);
++  Internal_EGifCloseFile(GifFile);
+ 
+   return true;
+ }
+@@ -165,17 +171,18 @@
+ {
+   GifFileType* GifFile;
+   GifByteType* Ptr;
++  int GifError;
+   
+-  if ((GifFile = EGifOpen (stream, &GIFOutputFunc)) == NULL)
++  if ((GifFile = EGifOpen (stream, &GIFOutputFunc, &GifError)) == NULL)
+     {
+-      std::cerr << "Error preparing GIF file for writing." << std::endl;
++      std::cerr << "Error preparing GIF file for writing: " << GifErrorString(GifError) << std::endl;
+       return false;
+     }
+   
+   int ColorMapSize = 256;
+   
+   // later use our own colormap generation
+-  ColorMapObject* OutputColorMap = MakeMapObject(ColorMapSize, NULL);
++  ColorMapObject* OutputColorMap = GifMakeMapObject(ColorMapSize, NULL);
+   if (!OutputColorMap)
+     return false;
+   
+@@ -203,7 +210,7 @@
+   }
+    
+   
+-  if (QuantizeBuffer(image.w, image.h, &ColorMapSize,
++  if (GifQuantizeBuffer(image.w, image.h, &ColorMapSize,
+ 		     RedBuffer, GreenBuffer, BlueBuffer,
+ 		     OutputBuffer, OutputColorMap->Colors) == GIF_ERROR) {
+     return false;
+@@ -215,7 +222,7 @@
+   if (EGifPutScreenDesc(GifFile, image.w, image.h,
+ 			ColorMapSize, 0, OutputColorMap) == GIF_ERROR ||
+       EGifPutImageDesc(GifFile, 0, 0, image.w, image.h,
+-		       FALSE, NULL) == GIF_ERROR)
++		       false, NULL) == GIF_ERROR)
+     {
+       std::cerr << "Error writing GIF header." << std::endl;
+       return false;
+@@ -234,7 +241,7 @@
+ 
+   delete (RedBuffer); delete (GreenBuffer); delete (BlueBuffer);
+ 
+-  EGifCloseFile(GifFile);
++  Internal_EGifCloseFile(GifFile);
+   return true;
+ }
+ 
================================================================

---- gitweb:

http://git.pld-linux.org/gitweb.cgi/packages/exact-image.git/commitdiff/e293edfcce9d1c4d57d74fea75fe7e0c53aefeb1



More information about the pld-cvs-commit mailing list