Review Request 110962: Switch to an external LibRaw

Pino Toscano pino at kde.org
Tue Jun 11 17:50:48 UTC 2013


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

Review request for KDE Graphics, Release Team and Gilles Caulier.


Description
-------

Instead of using an embedded copy of LibRaw, look for an external LibRaw as mandatory dependency with a new CMake module and using its variables.
    
Considering some LibRaw versions seem to be underlinked and not linking to OpenMP, link it manually in libkdcraw to overcome such lack.
    
Switch back to the MAKE_KDCRAW_LIB define (i.e. the default set by KDE4_ADD_LIBRARY) as the one used to check whether it is being built, as otherwise LIBRAW_BUILDLIB would conflict with LibRaw.

Once this RR is approved, I will remove the libraw code copy and the CMake modules (FindLCMS2.cmake and FindPthreads.cmake) needed for it.


This addresses bug 307146.
    http://bugs.kde.org/show_bug.cgi?id=307146


Diffs
-----

  CMakeLists.txt f2f269609feb10947ec3bac10125b379c6c821dd 
  cmake/modules/FindLibRaw.cmake PRE-CREATION 
  libkdcraw/CMakeLists.txt cce5d6dba690fb5182638ccd1f10488bbd6ec2ce 
  libkdcraw/libkdcraw_export.h 1a222a03502a0e068bdba4f03b7ff4961c4a8f2b 

Diff: http://git.reviewboard.kde.org/r/110962/diff/


Testing
-------

Compiles fine with both LibRaw 0.14.7 and 0.15.1.


Thanks,

Pino Toscano

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/release-team/attachments/20130611/170a8082/attachment.html>


More information about the release-team mailing list