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 760283 - Add libjasper as a sysdep so that gegl can load JPEG 2000
Add libjasper as a sysdep so that gegl can load JPEG 2000
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
Depends on:
Blocks:
 
 
Reported: 2016-01-07 17:29 UTC by Debarshi Ray
Modified: 2016-01-22 19:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
3.20: Add libjasper as a sysdep so that gegl can load JPEG 2000 (1.35 KB, patch)
2016-01-07 17:31 UTC, Debarshi Ray
committed Details | Review
3.20: Use a header file to detect libjasper (1000 bytes, patch)
2016-01-09 20:57 UTC, Ting-Wei Lan
committed Details | Review

Description Debarshi Ray 2016-01-07 17:29:26 UTC
gnome-photos sanity checks the presence of required gegl operations before showing a window to avoid weird behaviour at runtime. The JPEG 2000 source is one of them.

Since we recommend Fedora to newcomers, and Fedora's gtk3 package requires libjasper through gdk-pixbuf2-modules, I think it should be OK to add it.

[1] https://wiki.gnome.org/Newcomers/BuildGnome
Comment 1 Debarshi Ray 2016-01-07 17:31:14 UTC
Created attachment 318432 [details] [review]
3.20: Add libjasper as a sysdep so that gegl can load JPEG 2000
Comment 2 Javier Jardón (IRC: jjardon) 2016-01-07 17:59:57 UTC
Review of attachment 318432 [details] [review]:

Sure
Comment 3 Debarshi Ray 2016-01-07 18:23:45 UTC
Comment on attachment 318432 [details] [review]
3.20: Add libjasper as a sysdep so that gegl can load JPEG 2000

Thanks, jjardon. Pushed to master.
Comment 4 Ting-Wei Lan 2016-01-08 17:20:13 UTC
This doesn't work on all distributions because jasper upstream doesn't provide a .pc file. The jasper.pc file in Fedora package is added by a patch in SRPM.
Comment 5 Ting-Wei Lan 2016-01-09 20:57:14 UTC
Created attachment 318601 [details] [review]
3.20: Use a header file to detect libjasper

The upstream project does not provide jasper.pc file, so we cannot expect
it to be available in all distributions.
Comment 6 Michael Catanzaro 2016-01-22 19:06:29 UTC
Review of attachment 318601 [details] [review]:

Thanks.
Comment 7 Ting-Wei Lan 2016-01-22 19:07:35 UTC
Attachment 318601 [details] pushed as 6ef93b8 - 3.20: Use a header file to detect libjasper