After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 736328 - tracker-extract-gif: add compatibility with GIFLib 5.1.0
tracker-extract-gif: add compatibility with GIFLib 5.1.0
Status: RESOLVED DUPLICATE of bug 734351
Product: tracker
Classification: Core
Component: Extractor
unspecified
Other All
: Normal normal
: ---
Assigned To: tracker-extractor
Depends on:
Blocks:
 
 
Reported: 2014-09-09 13:12 UTC by Dominique Leuenberger
Modified: 2014-09-09 17:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
tracker-extract-gif: add compatibility with GIFLib 5.1.0 (1.34 KB, patch)
2014-09-09 13:12 UTC, Dominique Leuenberger
none Details | Review
Fix stupid typo (which worked with (1.34 KB, patch)
2014-09-09 13:30 UTC, Dominique Leuenberger
none Details | Review

Description Dominique Leuenberger 2014-09-09 13:12:30 UTC
GIFLib 5.1.0 had a minor API change impacting us.
Add compatibility to both APIs.
Comment 1 Dominique Leuenberger 2014-09-09 13:12:32 UTC
Created attachment 285739 [details] [review]
tracker-extract-gif: add compatibility with GIFLib 5.1.0

Signed-off-by: Dominique Leuenberger <dimstar@opensuse.org>
Comment 2 Dominique Leuenberger 2014-09-09 13:30:28 UTC
Created attachment 285741 [details] [review]
Fix stupid typo (which worked with
Comment 3 Martyn Russell 2014-09-09 17:00:53 UTC
Hi Dominique, interestingly there is already bug #734351 that has a patch for this. I've just not been able to test it.

I am going to mark this bug as a duplicate because it came after the other one.

Can I ask you to re-attach the patch to the other bug and continue discussion there? 

I would love to resolve this and I am happy to go on your word on this one if I can't test it.

Thanks again Dominique!

*** This bug has been marked as a duplicate of bug 734351 ***
Comment 4 Dominique Leuenberger 2014-09-09 17:07:10 UTC
Ups! Sorry... should have checked bugzilla, not only git :)

Will attach my patch there for discussion... seemed rather a simple patch.