Review Request 115120: Clean up the CMake files (and a couple of other bits)

Alex Merry kde at randomguy3.me.uk
Mon Jan 20 10:29:16 UTC 2014



> On Jan. 19, 2014, 8:07 p.m., David Faure wrote:
> > src/imageformats/CMakeLists.txt, line 53
> > <https://git.reviewboard.kde.org/r/115120/diff/1/?file=234669#file234669line53>
> >
> >     Remind me, who uses these desktop files?
> >     
> >     Only the kde4support code, right?

Yep.  I must get round to submitting that patch to Qt that should allow us to ditch them...


- Alex


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


On Jan. 19, 2014, 12:23 p.m., Alex Merry wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115120/
> -----------------------------------------------------------
> 
> (Updated Jan. 19, 2014, 12:23 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kimageformats
> 
> 
> Description
> -------
> 
> Clean up the CMake files (and a couple of other bits)
> 
> src/imageformats/CMakeLists.txt is now much clearer and more consistent.
> 
> Also, removed an unnecessary/unimplemented method from exr.cpp.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt c39d6f322d335fa5d19e934d0f7c5c602ba1a502 
>   cmake/FindOpenEXR.cmake b4ae656722a78e8f3494415e4583709e9b29065e 
>   src/imageformats/CMakeLists.txt 053054a54ef53b7695b244748c3e5c0f252cc8c3 
>   src/imageformats/config-kimgio.h.cmake  
>   src/imageformats/exr.h 3ae9e16e3d6e74240e14ce8994a1d9126c4bdd36 
>   src/imageformats/exr.cpp f8c70b7547781614b05936f893a526d12b4a41b1 
>   src/imageformats/jp2.cpp 5be5063d034b048aa47c5491796a89bc4519d3e4 
> 
> Diff: https://git.reviewboard.kde.org/r/115120/diff/
> 
> 
> Testing
> -------
> 
> Compiles, installs.
> 
> 
> Thanks,
> 
> Alex Merry
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140120/575dac18/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list