[Digikam-devel] Review Request 108821: fix libjpeg detection for libjpeg-turbo

Heena Mahour heena393 at gmail.com
Tue Feb 4 17:39:08 GMT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/108821/#review48955
-----------------------------------------------------------


>MACRO_LOG_FEATURE(JPEG_FOUND "libjpeg" "JPEG image format support" "http://www.ijg.org" FALSE "8" "needed for the LibRaw DNG lossy codec")
It should be MACRO_LOG_FEATURE(JPEG_FOUND "libjpeg" "JPEG image format support" "http://www.ijg.org" FALSE "80" "needed for the LibRaw DNG lossy codec").
Please remove whitespaces .

- Heena Mahour


On Feb. 6, 2013, 10:05 p.m., Kevin Kofler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/108821/
> -----------------------------------------------------------
> 
> (Updated Feb. 6, 2013, 10:05 p.m.)
> 
> 
> Review request for Digikam and KDE Graphics.
> 
> 
> Repository: libkdcraw
> 
> 
> Description
> -------
> 
> This fixes the libjpeg detection to check for the availability of jpeg_mem_src rather than for JPEG_LIB_VERSION, allowing to build the JPEG functionality with libjpeg-turbo >= 1.2.90.
> 
> RawSpeed/DngDecoderSlices.cpp tries to define its own jpeg_mem_src function if JPEG_LIB_VERSION < 80, so patch it to use the detected USE_JPEG8 instead to avoid the redefinition and the resulting compiler error.
> 
> Note: This patch is against 4.10, master is even more broken due to http://commits.kde.org/libkdcraw/016a3f26084b4bf7e0302ede362d937eb2dd4910 ("Fix compilation on Windows"), which unconditionally removes the USE_JPEG define, meaning JPEG support will never be built at all. This is clearly incorrect.
> 
> 
> Diffs
> -----
> 
>   libraw/CMakeLists.txt 581e7b1 
>   libraw/RawSpeed/DngDecoderSlices.cpp 28a137e 
>   libraw/RawSpeed/rawspeed.jpeg-version.patch PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/108821/diff/
> 
> 
> Testing
> -------
> 
> Builds (and detects jpeg_mem_src) in Rawhide (libkdcraw 4.10.0 release, libjpeg-turbo 20130204 snapshot (svn922, approx. 1.2.90)).
> 
> 
> Thanks,
> 
> Kevin Kofler
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20140204/eb7eaeff/attachment.html>


More information about the Digikam-devel mailing list