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

Albert Astals Cid aacid at kde.org
Tue May 7 18:11:41 BST 2013



> On May 7, 2013, 1:46 p.m., Gilles Caulier wrote:
> > Ship It!
> 
> Gilles Caulier wrote:
>     Problem fixed by Ananta by this commit :
>     
>     http://commits.kde.org/libkdcraw/908db171d4eba6989abb4e1a450e1d79c3c15d4e

Does really that one line commit do the same as Kevin's bigger commit? Shall this be marked as submitted then?


- Albert


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


On Feb. 6, 2013, 10:05 p.m., Kevin Kofler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108821/
> -----------------------------------------------------------
> 
> (Updated Feb. 6, 2013, 10:05 p.m.)
> 
> 
> Review request for Digikam and KDE Graphics.
> 
> 
> 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: http://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/20130507/cbf3d53d/attachment.html>


More information about the Digikam-devel mailing list