* [Buildroot] [PATCH 2/5] package/fbv: Switch to giflib
2015-01-26 19:24 [Buildroot] [PATCH 1/5] package/giflib: New package Bernd Kuhls
@ 2015-01-26 19:24 ` Bernd Kuhls
2015-01-27 15:17 ` Peter Korsgaard
2015-01-26 19:24 ` [Buildroot] [PATCH 3/5] package/imlib2: " Bernd Kuhls
` (3 subsequent siblings)
4 siblings, 1 reply; 23+ messages in thread
From: Bernd Kuhls @ 2015-01-26 19:24 UTC (permalink / raw)
To: buildroot
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
package/fbv/Config.in | 4 +--
package/fbv/fbv-1.0b-giflib.patch | 69 +++++++++++++++++++++++++++++++++++++
package/fbv/fbv.mk | 10 +++++-
3 files changed, 80 insertions(+), 3 deletions(-)
create mode 100644 package/fbv/fbv-1.0b-giflib.patch
diff --git a/package/fbv/Config.in b/package/fbv/Config.in
index b8a9fe4..3160a85 100644
--- a/package/fbv/Config.in
+++ b/package/fbv/Config.in
@@ -26,8 +26,8 @@ config BR2_PACKAGE_FBV_JPEG
config BR2_PACKAGE_FBV_GIF
bool "GIF support"
default y
- select BR2_PACKAGE_LIBUNGIF
+ select BR2_PACKAGE_GIFLIB
help
- Enable support for GIF using libungif.
+ Enable support for GIF using giflib.
endif # BR2_PACKAGE_FBV
diff --git a/package/fbv/fbv-1.0b-giflib.patch b/package/fbv/fbv-1.0b-giflib.patch
new file mode 100644
index 0000000..355a585
--- /dev/null
+++ b/package/fbv/fbv-1.0b-giflib.patch
@@ -0,0 +1,69 @@
+Adjust source code to work with giflib 5.1x
+
+Downloaded from
+https://projects.archlinux.org/svntogit/community.git/plain/trunk/giflib-5.1.patch?h=packages/fbv
+
+Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
+
+diff -wbBur fbv-1.0b/gif.c fbv-1.0b.my/gif.c
+--- fbv-1.0b/gif.c 2003-08-25 00:23:02.000000000 +0400
++++ fbv-1.0b.my/gif.c 2014-05-29 18:39:41.337332872 +0400
+@@ -31,10 +31,10 @@
+ #include <string.h>
+ #define min(a,b) ((a) < (b) ? (a) : (b))
+ #define gflush return(FH_ERROR_FILE);
+-#define grflush { DGifCloseFile(gft); return(FH_ERROR_FORMAT); }
+-#define mgrflush { free(lb); free(slb); DGifCloseFile(gft); return(FH_ERROR_FORMAT); }
++#define grflush { DGifCloseFile(gft, NULL); return(FH_ERROR_FORMAT); }
++#define mgrflush { free(lb); free(slb); DGifCloseFile(gft, NULL); return(FH_ERROR_FORMAT); }
+ #define agflush return(FH_ERROR_FORMAT);
+-#define agrflush { DGifCloseFile(gft); return(FH_ERROR_FORMAT); }
++#define agrflush { DGifCloseFile(gft, NULL); return(FH_ERROR_FORMAT); }
+
+
+ int fh_gif_id(char *name)
+@@ -81,7 +81,7 @@
+ ColorMapObject *cmap;
+ int cmaps;
+
+- gft=DGifOpenFileName(name);
++ gft=DGifOpenFileName(name, NULL);
+ if(gft==NULL){printf("err5\n"); gflush;} //////////
+ do
+ {
+@@ -170,7 +170,7 @@
+ }
+ }
+ while( rt!= TERMINATE_RECORD_TYPE );
+- DGifCloseFile(gft);
++ DGifCloseFile(gft, NULL);
+ return(FH_ERROR_OK);
+ }
+
+@@ -184,7 +184,7 @@
+ int extcode;
+ GifRecordType rt;
+
+- gft=DGifOpenFileName(name);
++ gft=DGifOpenFileName(name, NULL);
+ if(gft==NULL) gflush;
+ do
+ {
+@@ -197,7 +197,7 @@
+ px=gft->Image.Width;
+ py=gft->Image.Height;
+ *x=px; *y=py;
+- DGifCloseFile(gft);
++ DGifCloseFile(gft, NULL);
+ return(FH_ERROR_OK);
+ break;
+ case EXTENSION_RECORD_TYPE:
+@@ -210,7 +210,7 @@
+ }
+ }
+ while( rt!= TERMINATE_RECORD_TYPE );
+- DGifCloseFile(gft);
++ DGifCloseFile(gft, NULL);
+ return(FH_ERROR_FORMAT);
+ }
+ #endif
diff --git a/package/fbv/fbv.mk b/package/fbv/fbv.mk
index fe24a6f..81881ff 100644
--- a/package/fbv/fbv.mk
+++ b/package/fbv/fbv.mk
@@ -30,7 +30,15 @@ else
FBV_CONFIGURE_OPTS += --without-libjpeg
endif
ifeq ($(BR2_PACKAGE_FBV_GIF),y)
-FBV_DEPENDENCIES += libungif
+FBV_DEPENDENCIES += giflib
+define FBV_CONFIGURE_LIBUNGIF_IS_NOW_GIFLIB
+ $(SED) 's|-lungif|-lgif|' $(@D)/configure
+endef
+FBV_PRE_CONFIGURE_HOOKS += FBV_CONFIGURE_LIBUNGIF_IS_NOW_GIFLIB
+define FBV_MAKE_LIBUNGIF_IS_NOW_GIFLIB
+ $(SED) 's|-lungif|-lgif|' $(@D)/Make.conf
+endef
+FBV_POST_CONFIGURE_HOOKS += FBV_MAKE_LIBUNGIF_IS_NOW_GIFLIB
else
FBV_CONFIGURE_OPTS += --without-libungif
endif
--
1.7.10.4
^ permalink raw reply related [flat|nested] 23+ messages in thread* [Buildroot] [PATCH 2/5] package/fbv: Switch to giflib
2015-01-26 19:24 ` [Buildroot] [PATCH 2/5] package/fbv: Switch to giflib Bernd Kuhls
@ 2015-01-27 15:17 ` Peter Korsgaard
2015-01-29 18:56 ` Bernd Kuhls
0 siblings, 1 reply; 23+ messages in thread
From: Peter Korsgaard @ 2015-01-27 15:17 UTC (permalink / raw)
To: buildroot
>>>>> "Bernd" == Bernd Kuhls <bernd.kuhls@t-online.de> writes:
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/fbv/Config.in | 4 +--
> package/fbv/fbv-1.0b-giflib.patch | 69 +++++++++++++++++++++++++++++++++++++
> package/fbv/fbv.mk | 10 +++++-
> 3 files changed, 80 insertions(+), 3 deletions(-)
> create mode 100644 package/fbv/fbv-1.0b-giflib.patch
> diff --git a/package/fbv/fbv.mk b/package/fbv/fbv.mk
> index fe24a6f..81881ff 100644
> --- a/package/fbv/fbv.mk
> +++ b/package/fbv/fbv.mk
> @@ -30,7 +30,15 @@ else
> FBV_CONFIGURE_OPTS += --without-libjpeg
> endif
> ifeq ($(BR2_PACKAGE_FBV_GIF),y)
> -FBV_DEPENDENCIES += libungif
> +FBV_DEPENDENCIES += giflib
> +define FBV_CONFIGURE_LIBUNGIF_IS_NOW_GIFLIB
> + $(SED) 's|-lungif|-lgif|' $(@D)/configure
> +endef
> +FBV_PRE_CONFIGURE_HOOKS += FBV_CONFIGURE_LIBUNGIF_IS_NOW_GIFLIB
> +define FBV_MAKE_LIBUNGIF_IS_NOW_GIFLIB
> + $(SED) 's|-lungif|-lgif|' $(@D)/Make.conf
> +endef
Why is it needed to patch Make.conf if you already patch configure
(which generates it)?
I would prefer to see the configure changes in the giflib patch
instead. How do the archlinux guys handle it?
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 2/5] package/fbv: Switch to giflib
2015-01-27 15:17 ` Peter Korsgaard
@ 2015-01-29 18:56 ` Bernd Kuhls
0 siblings, 0 replies; 23+ messages in thread
From: Bernd Kuhls @ 2015-01-29 18:56 UTC (permalink / raw)
To: buildroot
Hi,
Peter Korsgaard <peter@korsgaard.com> wrote in
news:87a914z20n.fsf at dell.be.48ers.dk:
>>>>>> "Bernd" == Bernd Kuhls <bernd.kuhls-
zqRNUXuvxA0b1SvskN2V4Q@public.gmane.org> writes:
>
> > +define FBV_CONFIGURE_LIBUNGIF_IS_NOW_GIFLIB
> > + $(SED) 's|-lungif|-lgif|' $(@D)/configure
> > +endef
> > +FBV_PRE_CONFIGURE_HOOKS += FBV_CONFIGURE_LIBUNGIF_IS_NOW_GIFLIB
> > +define FBV_MAKE_LIBUNGIF_IS_NOW_GIFLIB
> > + $(SED) 's|-lungif|-lgif|' $(@D)/Make.conf
> > +endef
>
> Why is it needed to patch Make.conf if you already patch configure
> (which generates it)?
right, patching Make.conf is not necessary, I will remove it in the next
patch version.
> I would prefer to see the configure changes in the giflib patch
> instead. How do the archlinux guys handle it?
They are using sed to change Make.conf
https://projects.archlinux.org/svntogit/community.git/tree/trunk/PKGBUILD?
h=packages/fbv
which I did in the first place, but then configure printed this
checking for libungif presence... no
although compilation with libgif worked, so I patched the configure scripts
afterwards to fix the wrong configure log but forgot to remove the
Make.conf patch ;)
The next version will contain a patch for configure.
Regards, Bernd
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 3/5] package/imlib2: Switch to giflib
2015-01-26 19:24 [Buildroot] [PATCH 1/5] package/giflib: New package Bernd Kuhls
2015-01-26 19:24 ` [Buildroot] [PATCH 2/5] package/fbv: Switch to giflib Bernd Kuhls
@ 2015-01-26 19:24 ` Bernd Kuhls
2015-01-27 15:25 ` Peter Korsgaard
2015-01-26 19:24 ` [Buildroot] [PATCH 4/5] package/efl/libevas: " Bernd Kuhls
` (2 subsequent siblings)
4 siblings, 1 reply; 23+ messages in thread
From: Bernd Kuhls @ 2015-01-26 19:24 UTC (permalink / raw)
To: buildroot
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
package/imlib2/0001-giflib51.patch | 71 ++++++++++++++++++++++++++++++++++++
package/imlib2/Config.in | 2 +-
package/imlib2/imlib2.mk | 2 +-
3 files changed, 73 insertions(+), 2 deletions(-)
create mode 100644 package/imlib2/0001-giflib51.patch
diff --git a/package/imlib2/0001-giflib51.patch b/package/imlib2/0001-giflib51.patch
new file mode 100644
index 0000000..2b46329
--- /dev/null
+++ b/package/imlib2/0001-giflib51.patch
@@ -0,0 +1,71 @@
+Adjust source code to work with giflib 5.1x
+
+Downloaded from
+https://projects.archlinux.org/svntogit/packages.git/plain/trunk/imlib2-giflib51.patch?h=packages/imlib2
+
+Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
+
+diff -ruN imlib2-1.4.6.orig/src/modules/loaders/loader_gif.c imlib2-1.4.6/src/modules/loaders/loader_gif.c
+--- imlib2-1.4.6.orig/src/modules/loaders/loader_gif.c 2013-12-21 10:16:10.000000000 +0000
++++ imlib2-1.4.6/src/modules/loaders/loader_gif.c 2014-05-27 09:52:35.857291512 +0000
+@@ -36,7 +36,7 @@
+ #endif
+ if (fd < 0)
+ return 0;
+- gif = DGifOpenFileHandle(fd);
++ gif = DGifOpenFileHandle(fd, NULL);
+ if (!gif)
+ {
+ close(fd);
+@@ -60,13 +60,13 @@
+ h = gif->Image.Height;
+ if (!IMAGE_DIMENSIONS_OK(w, h))
+ {
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ return 0;
+ }
+ rows = malloc(h * sizeof(GifRowType *));
+ if (!rows)
+ {
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ return 0;
+ }
+ for (i = 0; i < h; i++)
+@@ -78,7 +78,7 @@
+ rows[i] = malloc(w * sizeof(GifPixelType));
+ if (!rows[i])
+ {
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ for (i = 0; i < h; i++)
+ {
+ if (rows[i])
+@@ -150,7 +150,7 @@
+ im->data = (DATA32 *) malloc(sizeof(DATA32) * w * h);
+ if (!im->data)
+ {
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ free(rows);
+ return 0;
+ }
+@@ -181,7 +181,7 @@
+ last_per = (int)per;
+ if (!(progress(im, (int)per, 0, last_y, w, i)))
+ {
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ for (i = 0; i < h; i++)
+ {
+ free(rows[i]);
+@@ -198,7 +198,7 @@
+ {
+ progress(im, 100, 0, last_y, w, h);
+ }
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ for (i = 0; i < h; i++)
+ {
+ free(rows[i]);
diff --git a/package/imlib2/Config.in b/package/imlib2/Config.in
index a323a8d..7aac074 100644
--- a/package/imlib2/Config.in
+++ b/package/imlib2/Config.in
@@ -23,7 +23,7 @@ config BR2_PACKAGE_IMLIB2_PNG
config BR2_PACKAGE_IMLIB2_GIF
bool "GIF support"
- select BR2_PACKAGE_LIBUNGIF
+ select BR2_PACKAGE_GIFLIB
config BR2_PACKAGE_IMLIB2_TIFF
bool "TIFF support"
diff --git a/package/imlib2/imlib2.mk b/package/imlib2/imlib2.mk
index 62f3989..d8feeb6 100644
--- a/package/imlib2/imlib2.mk
+++ b/package/imlib2/imlib2.mk
@@ -38,7 +38,7 @@ endif
ifeq ($(BR2_PACKAGE_IMLIB2_GIF),y)
IMLIB2_CONF_OPTS += --with-gif
- IMLIB2_DEPENDENCIES += libungif
+ IMLIB2_DEPENDENCIES += giflib
else
IMLIB2_CONF_OPTS += --without-gif
endif
--
1.7.10.4
^ permalink raw reply related [flat|nested] 23+ messages in thread* [Buildroot] [PATCH 3/5] package/imlib2: Switch to giflib
2015-01-26 19:24 ` [Buildroot] [PATCH 3/5] package/imlib2: " Bernd Kuhls
@ 2015-01-27 15:25 ` Peter Korsgaard
2015-01-29 19:13 ` Bernd Kuhls
0 siblings, 1 reply; 23+ messages in thread
From: Peter Korsgaard @ 2015-01-27 15:25 UTC (permalink / raw)
To: buildroot
>>>>> "Bernd" == Bernd Kuhls <bernd.kuhls@t-online.de> writes:
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/imlib2/0001-giflib51.patch | 71 ++++++++++++++++++++++++++++++++++++
> package/imlib2/Config.in | 2 +-
> package/imlib2/imlib2.mk | 2 +-
> 3 files changed, 73 insertions(+), 2 deletions(-)
> create mode 100644 package/imlib2/0001-giflib51.patch
How about instead bumping our imlib2 version? I see there is some giflib
support in upstream git:
https://git.enlightenment.org/legacy/imlib2.git/log/
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 3/5] package/imlib2: Switch to giflib
2015-01-27 15:25 ` Peter Korsgaard
@ 2015-01-29 19:13 ` Bernd Kuhls
2015-03-07 18:09 ` Fabio Porcedda
0 siblings, 1 reply; 23+ messages in thread
From: Bernd Kuhls @ 2015-01-29 19:13 UTC (permalink / raw)
To: buildroot
[posted and mailed]
Peter Korsgaard <peter@korsgaard.com> wrote in
news:8761bsz1n6.fsf at dell.be.48ers.dk:
> How about instead bumping our imlib2 version? I see there is some giflib
> support in upstream git:
>
> https://git.enlightenment.org/legacy/imlib2.git/log/
Hi,
bumping to 1.4.6 and backporting some giflib related patches or switching
directly to the upstream git HEAD is an option, but I am not in a position to
runtime test the packages depending on imlib2, therefore I chose a way to
support giflib with a minimum of code changes. I can provide v2 of my patch
with one of my suggestions but fixing build errors later on will be a
difficult task for me timewise. What do you suggest?
Since imlib2 1.4.6 was released over a year ago I suppose that there is not a
big audience for this package in the buildroot community, the only reason I
touched imlib2 is my desire to get rid of libungif in favour of giflib
because Kodi 15.0 will depend on it ;)
Regards, Bernd
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 3/5] package/imlib2: Switch to giflib
2015-01-29 19:13 ` Bernd Kuhls
@ 2015-03-07 18:09 ` Fabio Porcedda
0 siblings, 0 replies; 23+ messages in thread
From: Fabio Porcedda @ 2015-03-07 18:09 UTC (permalink / raw)
To: buildroot
On Thu, Jan 29, 2015 at 8:13 PM, Bernd Kuhls <bernd.kuhls@t-online.de> wrote:
> [posted and mailed]
>
> Peter Korsgaard <peter@korsgaard.com> wrote in
> news:8761bsz1n6.fsf at dell.be.48ers.dk:
>
>> How about instead bumping our imlib2 version? I see there is some giflib
>> support in upstream git:
>>
>> https://git.enlightenment.org/legacy/imlib2.git/log/
>
> Hi,
>
> bumping to 1.4.6 and backporting some giflib related patches or switching
> directly to the upstream git HEAD is an option, but I am not in a position to
> runtime test the packages depending on imlib2, therefore I chose a way to
> support giflib with a minimum of code changes. I can provide v2 of my patch
> with one of my suggestions but fixing build errors later on will be a
> difficult task for me timewise. What do you suggest?
>
> Since imlib2 1.4.6 was released over a year ago I suppose that there is not a
> big audience for this package in the buildroot community, the only reason I
> touched imlib2 is my desire to get rid of libungif in favour of giflib
> because Kodi 15.0 will depend on it ;)
I've made a patch for bumping it to 1.4.6 and back porting few
upstream patches to support giflib:
http://patchwork.ozlabs.org/patch/447618/
http://patchwork.ozlabs.org/patch/447619/
Sorry but I've found your patch only right before sending mine.
BR
--
Fabio Porcedda
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 4/5] package/efl/libevas: Switch to giflib
2015-01-26 19:24 [Buildroot] [PATCH 1/5] package/giflib: New package Bernd Kuhls
2015-01-26 19:24 ` [Buildroot] [PATCH 2/5] package/fbv: Switch to giflib Bernd Kuhls
2015-01-26 19:24 ` [Buildroot] [PATCH 3/5] package/imlib2: " Bernd Kuhls
@ 2015-01-26 19:24 ` Bernd Kuhls
2015-03-07 16:29 ` Fabio Porcedda
2015-04-04 16:45 ` Thomas Petazzoni
2015-01-26 19:24 ` [Buildroot] [PATCH 5/5] package/libungif: deprecate package Bernd Kuhls
2015-01-27 15:12 ` [Buildroot] [PATCH 1/5] package/giflib: New package Peter Korsgaard
4 siblings, 2 replies; 23+ messages in thread
From: Bernd Kuhls @ 2015-01-26 19:24 UTC (permalink / raw)
To: buildroot
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
package/efl/libevas/0001-giflib.patch | 91 +++++++++++++++++++++++++++++++++
package/efl/libevas/Config.in | 2 +-
package/efl/libevas/libevas.mk | 2 +-
3 files changed, 93 insertions(+), 2 deletions(-)
create mode 100644 package/efl/libevas/0001-giflib.patch
diff --git a/package/efl/libevas/0001-giflib.patch b/package/efl/libevas/0001-giflib.patch
new file mode 100644
index 0000000..1176721
--- /dev/null
+++ b/package/efl/libevas/0001-giflib.patch
@@ -0,0 +1,91 @@
+Adjust source code to work with giflib 5.1x
+
+Downloaded from
+http://commit.cvs.pld.groups.com.ru/332837-packagesevas_fix_building_with_giflib_51_rel_6
+
+Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
+
+diff -uNr evas-1.7.10.org/src/modules/loaders/gif/evas_image_load_gif.c evas-1.7.10/src/modules/loaders/gif/evas_image_load_gif.c
+--- evas-1.7.10.org/src/modules/loaders/gif/evas_image_load_gif.c 2013-08-01 17:41:35.000000000 +0200
++++ evas-1.7.10/src/modules/loaders/gif/evas_image_load_gif.c 2015-01-26 19:44:20.132955194 +0100
+@@ -338,7 +338,7 @@
+
+ if (!cmap)
+ {
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ for (i = 0; i < scale_h; i++)
+ {
+ free(rows[i]);
+@@ -725,7 +725,7 @@
+ if ((w < 1) || (h < 1) || (w > IMG_MAX_SIZE) || (h > IMG_MAX_SIZE) ||
+ IMG_TOO_BIG(w, h))
+ {
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ if (IMG_TOO_BIG(w, h))
+ *error = EVAS_LOAD_ERROR_RESOURCE_ALLOCATION_FAILED;
+ else
+@@ -740,7 +740,7 @@
+ if (DGifGetRecordType(gif, &rec) == GIF_ERROR)
+ {
+ /* PrintGifError(); */
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ *error = EVAS_LOAD_ERROR_UNKNOWN_FORMAT;
+ return EINA_FALSE;
+ }
+@@ -754,7 +754,7 @@
+ if (DGifGetImageDesc(gif) == GIF_ERROR)
+ {
+ /* PrintGifError(); */
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ *error = EVAS_LOAD_ERROR_UNKNOWN_FORMAT;
+ return EINA_FALSE;
+ }
+@@ -762,7 +762,7 @@
+ if (DGifGetCode(gif, &img_code, &img) == GIF_ERROR)
+ {
+ /* PrintGifError(); */
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ *error = EVAS_LOAD_ERROR_UNKNOWN_FORMAT;
+ return EINA_FALSE;
+ }
+@@ -818,7 +818,7 @@
+ ie->frames = NULL;
+ }
+
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ *error = EVAS_LOAD_ERROR_NONE;
+ return EINA_TRUE;
+ }
+@@ -885,7 +885,7 @@
+ }
+
+ ie->frames = eina_list_append(ie->frames, frame);
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ return EINA_TRUE;
+ }
+
+@@ -959,7 +959,7 @@
+ *error = EVAS_LOAD_ERROR_UNKNOWN_FORMAT;
+ return EINA_FALSE;
+ }
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ *error = EVAS_LOAD_ERROR_NONE;
+ return EINA_TRUE;
+ }
+@@ -1080,7 +1080,7 @@
+ }
+ } while (rec != TERMINATE_RECORD_TYPE);
+
+- DGifCloseFile(gif);
++ DGifCloseFile(gif, NULL);
+ return duration;
+ }
+
diff --git a/package/efl/libevas/Config.in b/package/efl/libevas/Config.in
index febc115..fa36e80 100644
--- a/package/efl/libevas/Config.in
+++ b/package/efl/libevas/Config.in
@@ -177,7 +177,7 @@ config BR2_PACKAGE_LIBEVAS_JPEG
config BR2_PACKAGE_LIBEVAS_GIF
bool "libevas gif loader"
- select BR2_PACKAGE_LIBUNGIF
+ select BR2_PACKAGE_GIFLIB
help
This enables the loader code that loads gif files using
libungif.
diff --git a/package/efl/libevas/libevas.mk b/package/efl/libevas/libevas.mk
index b5a2d72..9f2e801 100644
--- a/package/efl/libevas/libevas.mk
+++ b/package/efl/libevas/libevas.mk
@@ -167,7 +167,7 @@ endif
ifeq ($(BR2_PACKAGE_LIBEVAS_GIF),y)
LIBEVAS_CONF_OPTS += --enable-image-loader-gif
-LIBEVAS_DEPENDENCIES += libungif
+LIBEVAS_DEPENDENCIES += giflib
else
LIBEVAS_CONF_OPTS += --disable-image-loader-gif
endif
--
1.7.10.4
^ permalink raw reply related [flat|nested] 23+ messages in thread* [Buildroot] [PATCH 4/5] package/efl/libevas: Switch to giflib
2015-01-26 19:24 ` [Buildroot] [PATCH 4/5] package/efl/libevas: " Bernd Kuhls
@ 2015-03-07 16:29 ` Fabio Porcedda
2015-03-07 16:32 ` Thomas Petazzoni
2015-04-04 16:45 ` Thomas Petazzoni
1 sibling, 1 reply; 23+ messages in thread
From: Fabio Porcedda @ 2015-03-07 16:29 UTC (permalink / raw)
To: buildroot
On Mon, Jan 26, 2015 at 8:24 PM, Bernd Kuhls <bernd.kuhls@t-online.de> wrote:
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/efl/libevas/0001-giflib.patch | 91 +++++++++++++++++++++++++++++++++
> package/efl/libevas/Config.in | 2 +-
> package/efl/libevas/libevas.mk | 2 +-
> 3 files changed, 93 insertions(+), 2 deletions(-)
> create mode 100644 package/efl/libevas/0001-giflib.patch
>
> diff --git a/package/efl/libevas/0001-giflib.patch b/package/efl/libevas/0001-giflib.patch
> new file mode 100644
> index 0000000..1176721
> --- /dev/null
> +++ b/package/efl/libevas/0001-giflib.patch
> @@ -0,0 +1,91 @@
> +Adjust source code to work with giflib 5.1x
> +
> +Downloaded from
> +http://commit.cvs.pld.groups.com.ru/332837-packagesevas_fix_building_with_giflib_51_rel_6
> +
> +Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> +
> +diff -uNr evas-1.7.10.org/src/modules/loaders/gif/evas_image_load_gif.c evas-1.7.10/src/modules/loaders/gif/evas_image_load_gif.c
> +--- evas-1.7.10.org/src/modules/loaders/gif/evas_image_load_gif.c 2013-08-01 17:41:35.000000000 +0200
> ++++ evas-1.7.10/src/modules/loaders/gif/evas_image_load_gif.c 2015-01-26 19:44:20.132955194 +0100
> +@@ -338,7 +338,7 @@
> +
> + if (!cmap)
> + {
> +- DGifCloseFile(gif);
> ++ DGifCloseFile(gif, NULL);
> + for (i = 0; i < scale_h; i++)
> + {
> + free(rows[i]);
> +@@ -725,7 +725,7 @@
> + if ((w < 1) || (h < 1) || (w > IMG_MAX_SIZE) || (h > IMG_MAX_SIZE) ||
> + IMG_TOO_BIG(w, h))
> + {
> +- DGifCloseFile(gif);
> ++ DGifCloseFile(gif, NULL);
IMHO it's cleaner if instead of adding this patch you can add a
post_patch hook that using sed change all "DGifCloseFile(gif)" to
"DGifCloseFile(gif, NULL)".
> + if (IMG_TOO_BIG(w, h))
> + *error = EVAS_LOAD_ERROR_RESOURCE_ALLOCATION_FAILED;
> + else
> +@@ -740,7 +740,7 @@
> + if (DGifGetRecordType(gif, &rec) == GIF_ERROR)
> + {
> + /* PrintGifError(); */
> +- DGifCloseFile(gif);
> ++ DGifCloseFile(gif, NULL);
> + *error = EVAS_LOAD_ERROR_UNKNOWN_FORMAT;
> + return EINA_FALSE;
> + }
> +@@ -754,7 +754,7 @@
> + if (DGifGetImageDesc(gif) == GIF_ERROR)
> + {
> + /* PrintGifError(); */
> +- DGifCloseFile(gif);
> ++ DGifCloseFile(gif, NULL);
> + *error = EVAS_LOAD_ERROR_UNKNOWN_FORMAT;
> + return EINA_FALSE;
> + }
> +@@ -762,7 +762,7 @@
> + if (DGifGetCode(gif, &img_code, &img) == GIF_ERROR)
> + {
> + /* PrintGifError(); */
> +- DGifCloseFile(gif);
> ++ DGifCloseFile(gif, NULL);
> + *error = EVAS_LOAD_ERROR_UNKNOWN_FORMAT;
> + return EINA_FALSE;
> + }
> +@@ -818,7 +818,7 @@
> + ie->frames = NULL;
> + }
> +
> +- DGifCloseFile(gif);
> ++ DGifCloseFile(gif, NULL);
> + *error = EVAS_LOAD_ERROR_NONE;
> + return EINA_TRUE;
> + }
> +@@ -885,7 +885,7 @@
> + }
> +
> + ie->frames = eina_list_append(ie->frames, frame);
> +- DGifCloseFile(gif);
> ++ DGifCloseFile(gif, NULL);
> + return EINA_TRUE;
> + }
> +
> +@@ -959,7 +959,7 @@
> + *error = EVAS_LOAD_ERROR_UNKNOWN_FORMAT;
> + return EINA_FALSE;
> + }
> +- DGifCloseFile(gif);
> ++ DGifCloseFile(gif, NULL);
> + *error = EVAS_LOAD_ERROR_NONE;
> + return EINA_TRUE;
> + }
> +@@ -1080,7 +1080,7 @@
> + }
> + } while (rec != TERMINATE_RECORD_TYPE);
> +
> +- DGifCloseFile(gif);
> ++ DGifCloseFile(gif, NULL);
> + return duration;
> + }
> +
> diff --git a/package/efl/libevas/Config.in b/package/efl/libevas/Config.in
> index febc115..fa36e80 100644
> --- a/package/efl/libevas/Config.in
> +++ b/package/efl/libevas/Config.in
> @@ -177,7 +177,7 @@ config BR2_PACKAGE_LIBEVAS_JPEG
>
> config BR2_PACKAGE_LIBEVAS_GIF
> bool "libevas gif loader"
> - select BR2_PACKAGE_LIBUNGIF
> + select BR2_PACKAGE_GIFLIB
> help
> This enables the loader code that loads gif files using
> libungif.
> diff --git a/package/efl/libevas/libevas.mk b/package/efl/libevas/libevas.mk
> index b5a2d72..9f2e801 100644
> --- a/package/efl/libevas/libevas.mk
> +++ b/package/efl/libevas/libevas.mk
> @@ -167,7 +167,7 @@ endif
>
> ifeq ($(BR2_PACKAGE_LIBEVAS_GIF),y)
> LIBEVAS_CONF_OPTS += --enable-image-loader-gif
> -LIBEVAS_DEPENDENCIES += libungif
> +LIBEVAS_DEPENDENCIES += giflib
> else
> LIBEVAS_CONF_OPTS += --disable-image-loader-gif
> endif
> --
> 1.7.10.4
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
Fabio Porcedda
^ permalink raw reply [flat|nested] 23+ messages in thread* [Buildroot] [PATCH 4/5] package/efl/libevas: Switch to giflib
2015-03-07 16:29 ` Fabio Porcedda
@ 2015-03-07 16:32 ` Thomas Petazzoni
2015-03-07 17:01 ` Fabio Porcedda
0 siblings, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2015-03-07 16:32 UTC (permalink / raw)
To: buildroot
Dear Fabio Porcedda,
On Sat, 7 Mar 2015 17:29:56 +0100, Fabio Porcedda wrote:
> IMHO it's cleaner if instead of adding this patch you can add a
> post_patch hook that using sed change all "DGifCloseFile(gif)" to
> "DGifCloseFile(gif, NULL)".
I don't agree. Such sed expressions can very easily be overlooked when
bumping the package, and they don't solve a problem that should be
solved by submitting a patch to the upstream project.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 4/5] package/efl/libevas: Switch to giflib
2015-03-07 16:32 ` Thomas Petazzoni
@ 2015-03-07 17:01 ` Fabio Porcedda
2015-03-08 7:15 ` Thomas Petazzoni
0 siblings, 1 reply; 23+ messages in thread
From: Fabio Porcedda @ 2015-03-07 17:01 UTC (permalink / raw)
To: buildroot
On Sat, Mar 7, 2015 at 5:32 PM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Dear Fabio Porcedda,
>
> On Sat, 7 Mar 2015 17:29:56 +0100, Fabio Porcedda wrote:
>
>> IMHO it's cleaner if instead of adding this patch you can add a
>> post_patch hook that using sed change all "DGifCloseFile(gif)" to
>> "DGifCloseFile(gif, NULL)".
>
> I don't agree. Such sed expressions can very easily be overlooked when
> bumping the package, and they don't solve a problem that should be
> solved by submitting a patch to the upstream project.
I don't know exactly but the libevas seems dead, the last change in
the git repository was made 15 months ago.
https://git.enlightenment.org/legacy/evas.git/
I hope there is a chance for the patch to be accepted.
Bernd, have you tried to send the patch upstream?
Could you add a note in patch to justify the change?
Example:
From http://giflib.sourceforge.net/gif_lib.html
GIF file openers and closers - DGifOpenFileName(),
DGifOpenFileHandle(), DGifOpen(), DGifClose(), EGifOpenFileName(),
EGifOpenFileHandle(), EGifOpen(), and EGifClose() - all now take a
final integer address argument. If non-null, this is used to pass back
an error code when the function returns NULL.
With above note added and because the sed option is not available:
Acked-by: Fabio Porcedda <fabio.porcedda@gmail.com>
Tested-by: Fabio Porcedda <fabio.porcedda@gmail.com>
[build fine the following defconfig:
BR2_TOOLCHAIN_EXTERNAL=y
BR2_PACKAGE_EFL=y
BR2_PACKAGE_LIBEVAS=y
BR2_PACKAGE_LIBEVAS_GIF=y]
BR
--
Fabio Porcedda
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 4/5] package/efl/libevas: Switch to giflib
2015-03-07 17:01 ` Fabio Porcedda
@ 2015-03-08 7:15 ` Thomas Petazzoni
2015-03-08 7:28 ` Fabio Porcedda
0 siblings, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2015-03-08 7:15 UTC (permalink / raw)
To: buildroot
Dear Fabio Porcedda,
On Sat, 7 Mar 2015 18:01:41 +0100, Fabio Porcedda wrote:
> I don't know exactly but the libevas seems dead, the last change in
> the git repository was made 15 months ago.
The EFL stuff in Buildroot needs a complete update: most of the
EFL libraries have been merged into one single project/tarball. So many
of the EFL related packages in Buildroot should be removed, and the
main EFL library bumped to a higher version.
> https://git.enlightenment.org/legacy/evas.git/
See the "legacy" in this URL ?
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 4/5] package/efl/libevas: Switch to giflib
2015-03-08 7:15 ` Thomas Petazzoni
@ 2015-03-08 7:28 ` Fabio Porcedda
2015-03-08 7:46 ` Thomas Petazzoni
0 siblings, 1 reply; 23+ messages in thread
From: Fabio Porcedda @ 2015-03-08 7:28 UTC (permalink / raw)
To: buildroot
On Sun, Mar 8, 2015 at 8:15 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Dear Fabio Porcedda,
>
> On Sat, 7 Mar 2015 18:01:41 +0100, Fabio Porcedda wrote:
>
>> I don't know exactly but the libevas seems dead, the last change in
>> the git repository was made 15 months ago.
>
> The EFL stuff in Buildroot needs a complete update: most of the
> EFL libraries have been merged into one single project/tarball. So many
> of the EFL related packages in Buildroot should be removed, and the
> main EFL library bumped to a higher version.
>
>> https://git.enlightenment.org/legacy/evas.git/
>
> See the "legacy" in this URL ?
I understand that is a legacy repository but because you talked about
to send a patch upstream i was trying to understand if there is any
chance that the patch will be accepted.
So when you have talked about to send the patch upstream didn't know
about the legacy status of this repository?
Nevertheless the https://git.enlightenment.org/legacy/imlib2.git/
repository gets updated even if it has the "legacy" word in the
prefix, but it's the only one so maybe it's just an exception.
BR
--
Fabio Porcedda
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 4/5] package/efl/libevas: Switch to giflib
2015-03-08 7:28 ` Fabio Porcedda
@ 2015-03-08 7:46 ` Thomas Petazzoni
2015-03-08 7:54 ` Fabio Porcedda
0 siblings, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2015-03-08 7:46 UTC (permalink / raw)
To: buildroot
Dear Fabio Porcedda,
On Sun, 8 Mar 2015 08:28:44 +0100, Fabio Porcedda wrote:
> I understand that is a legacy repository but because you talked about
> to send a patch upstream i was trying to understand if there is any
> chance that the patch will be accepted.
>
> So when you have talked about to send the patch upstream didn't know
> about the legacy status of this repository?
Well, when you want to send a patch upstream, then obviously you have
to send the patch based on the latest upstream version. Which generally
involves bumping the Buildroot package to use the latest stable version
from upstream.
In the case of EFL, it indeed involves quite a bit of work, because all
the libraries that used to be split in separate tarballs are now
provided as one single tarball by upstream.
See https://phab.enlightenment.org/phame/live/3/post/efl_1_8/ :
"""
We have merged Ecore, Edje, Eet, Eeze, Efreet, Eina, Eio, Embryo,
Emotion, Ethumb, Evas and Evil into a single EFL package (as you can
see above these are no longer there, but inside the EFL distribution).
We also added in Eldbus, EPhysics, and Escape.
"""
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 4/5] package/efl/libevas: Switch to giflib
2015-03-08 7:46 ` Thomas Petazzoni
@ 2015-03-08 7:54 ` Fabio Porcedda
2015-03-08 8:03 ` Thomas Petazzoni
0 siblings, 1 reply; 23+ messages in thread
From: Fabio Porcedda @ 2015-03-08 7:54 UTC (permalink / raw)
To: buildroot
On Sun, Mar 8, 2015 at 8:46 AM, Thomas Petazzoni
<thomas.petazzoni@free-electrons.com> wrote:
> Dear Fabio Porcedda,
>
> On Sun, 8 Mar 2015 08:28:44 +0100, Fabio Porcedda wrote:
>
>> I understand that is a legacy repository but because you talked about
>> to send a patch upstream i was trying to understand if there is any
>> chance that the patch will be accepted.
>>
>> So when you have talked about to send the patch upstream didn't know
>> about the legacy status of this repository?
>
> Well, when you want to send a patch upstream, then obviously you have
> to send the patch based on the latest upstream version. Which generally
> involves bumping the Buildroot package to use the latest stable version
> from upstream.
>
> In the case of EFL, it indeed involves quite a bit of work, because all
> the libraries that used to be split in separate tarballs are now
> provided as one single tarball by upstream.
>
> See https://phab.enlightenment.org/phame/live/3/post/efl_1_8/ :
>
> """
> We have merged Ecore, Edje, Eet, Eeze, Efreet, Eina, Eio, Embryo,
> Emotion, Ethumb, Evas and Evil into a single EFL package (as you can
> see above these are no longer there, but inside the EFL distribution).
> We also added in Eldbus, EPhysics, and Escape.
> """
In the efl project where the evas library was merged, they have
already added support for giflib 5:
https://git.enlightenment.org/core/efl.git/commit/?id=096ae0e86fe306d40d744c8c11960dd9e736c4b4
So maybe we can just back port that commit.
BR
--
Fabio Porcedda
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 4/5] package/efl/libevas: Switch to giflib
2015-03-08 7:54 ` Fabio Porcedda
@ 2015-03-08 8:03 ` Thomas Petazzoni
0 siblings, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2015-03-08 8:03 UTC (permalink / raw)
To: buildroot
Dear Fabio Porcedda,
On Sun, 8 Mar 2015 08:54:52 +0100, Fabio Porcedda wrote:
> In the efl project where the evas library was merged, they have
> already added support for giflib 5:
> https://git.enlightenment.org/core/efl.git/commit/?id=096ae0e86fe306d40d744c8c11960dd9e736c4b4
>
> So maybe we can just back port that commit.
If you bump EFL to the latest version, there's no need to do this,
since the commit is already part of EFL releases.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 4/5] package/efl/libevas: Switch to giflib
2015-01-26 19:24 ` [Buildroot] [PATCH 4/5] package/efl/libevas: " Bernd Kuhls
2015-03-07 16:29 ` Fabio Porcedda
@ 2015-04-04 16:45 ` Thomas Petazzoni
1 sibling, 0 replies; 23+ messages in thread
From: Thomas Petazzoni @ 2015-04-04 16:45 UTC (permalink / raw)
To: buildroot
Dear Bernd Kuhls,
On Mon, 26 Jan 2015 20:24:19 +0100, Bernd Kuhls wrote:
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/efl/libevas/0001-giflib.patch | 91 +++++++++++++++++++++++++++++++++
> package/efl/libevas/Config.in | 2 +-
> package/efl/libevas/libevas.mk | 2 +-
> 3 files changed, 93 insertions(+), 2 deletions(-)
> create mode 100644 package/efl/libevas/0001-giflib.patch
Thanks, applied.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 5/5] package/libungif: deprecate package
2015-01-26 19:24 [Buildroot] [PATCH 1/5] package/giflib: New package Bernd Kuhls
` (2 preceding siblings ...)
2015-01-26 19:24 ` [Buildroot] [PATCH 4/5] package/efl/libevas: " Bernd Kuhls
@ 2015-01-26 19:24 ` Bernd Kuhls
2015-03-07 17:19 ` Fabio Porcedda
2015-04-04 16:46 ` Thomas Petazzoni
2015-01-27 15:12 ` [Buildroot] [PATCH 1/5] package/giflib: New package Peter Korsgaard
4 siblings, 2 replies; 23+ messages in thread
From: Bernd Kuhls @ 2015-01-26 19:24 UTC (permalink / raw)
To: buildroot
To be replaced with giflib.
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
package/libungif/Config.in | 2 ++
1 file changed, 2 insertions(+)
diff --git a/package/libungif/Config.in b/package/libungif/Config.in
index e8c301c..012ef76 100644
--- a/package/libungif/Config.in
+++ b/package/libungif/Config.in
@@ -1,5 +1,7 @@
config BR2_PACKAGE_LIBUNGIF
bool "libungif"
+ depends on !BR2_PACKAGE_GIFLIB
+ depends on BR2_DEPRECATED_SINCE_2015_02 # giflib
help
libungif is a library for handling the uncompressed GIF image format.
--
1.7.10.4
^ permalink raw reply related [flat|nested] 23+ messages in thread* [Buildroot] [PATCH 5/5] package/libungif: deprecate package
2015-01-26 19:24 ` [Buildroot] [PATCH 5/5] package/libungif: deprecate package Bernd Kuhls
@ 2015-03-07 17:19 ` Fabio Porcedda
2015-04-04 16:46 ` Thomas Petazzoni
1 sibling, 0 replies; 23+ messages in thread
From: Fabio Porcedda @ 2015-03-07 17:19 UTC (permalink / raw)
To: buildroot
On Mon, Jan 26, 2015 at 8:24 PM, Bernd Kuhls <bernd.kuhls@t-online.de> wrote:
> To be replaced with giflib.
Can you respin this patch?
It's important to deprecate libungif because it's incompatible with
libgif so if both are installed, the headers get overwritten, right
now we don't have build problems because we are lucky and they are
overwritten in right order :)
But using top-parallel make i get build failures because the order is changed.
Example:
BR2_TOOLCHAIN_EXTERNAL=y
BR2_PACKAGE_FBV=y
BR2_PACKAGE_LIBUNGIF=y
The following command build fine:
make clean libungif giflib fbv
But fbv fail to build with the following command:
make clean giflib libungif fbv
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/libungif/Config.in | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/package/libungif/Config.in b/package/libungif/Config.in
> index e8c301c..012ef76 100644
> --- a/package/libungif/Config.in
> +++ b/package/libungif/Config.in
> @@ -1,5 +1,7 @@
> config BR2_PACKAGE_LIBUNGIF
> bool "libungif"
> + depends on !BR2_PACKAGE_GIFLIB
> + depends on BR2_DEPRECATED_SINCE_2015_02 # giflib
> help
> libungif is a library for handling the uncompressed GIF image format.
>
> --
> 1.7.10.4
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
Fabio Porcedda
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 5/5] package/libungif: deprecate package
2015-01-26 19:24 ` [Buildroot] [PATCH 5/5] package/libungif: deprecate package Bernd Kuhls
2015-03-07 17:19 ` Fabio Porcedda
@ 2015-04-04 16:46 ` Thomas Petazzoni
2015-04-08 20:33 ` Bernd Kuhls
1 sibling, 1 reply; 23+ messages in thread
From: Thomas Petazzoni @ 2015-04-04 16:46 UTC (permalink / raw)
To: buildroot
Dear Bernd Kuhls,
On Mon, 26 Jan 2015 20:24:20 +0100, Bernd Kuhls wrote:
> To be replaced with giflib.
>
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
> package/libungif/Config.in | 2 ++
> 1 file changed, 2 insertions(+)
I cannot apply this patch: Kodi is still selecting libungif currently.
Can you resend an updated patch series that fixes Kodi before marking
libungif as deprecated ?
I'll mark this patch as Changes Requested in patchwork in the mean time.
Thanks,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Buildroot] [PATCH 1/5] package/giflib: New package
2015-01-26 19:24 [Buildroot] [PATCH 1/5] package/giflib: New package Bernd Kuhls
` (3 preceding siblings ...)
2015-01-26 19:24 ` [Buildroot] [PATCH 5/5] package/libungif: deprecate package Bernd Kuhls
@ 2015-01-27 15:12 ` Peter Korsgaard
4 siblings, 0 replies; 23+ messages in thread
From: Peter Korsgaard @ 2015-01-27 15:12 UTC (permalink / raw)
To: buildroot
>>>>> "Bernd" == Bernd Kuhls <bernd.kuhls@t-online.de> writes:
> This package will replace libungif.
> Needed for Kodi 15.0, TexturePacker depends on giflib >= 5.0.5
> https://github.com/xbmc/xbmc/blob/master/tools/depends/native/TexturePacker/src/configure.ac#L29
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
Committed, thanks.
I did a quick test and they are basically equivalent in file size:
find output/staging/usr/lib/lib*gif.so* -type f | xargs ls -lah
-rwxr-xr-x 1 peko peko 31K Jan 27 16:04 output/staging/usr/lib/libgif.so.7.0.0
-rwxr-xr-x 1 peko peko 32K Jan 27 16:08 output/staging/usr/lib/libungif.so.4.1.4
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 23+ messages in thread