Review Request: ksplashx: Adjust erroneous CMake calls.

Commit Hook null at kde.org
Mon Jul 23 03:05:48 UTC 2012


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


This review has been submitted with commit 7d11d3a09b981201f0bd8c2404f208547331f55c by Raphael Kubo da Costa to branch master.

- Commit Hook


On July 7, 2012, 3:07 a.m., Raphael Kubo da Costa wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105468/
> -----------------------------------------------------------
> 
> (Updated July 7, 2012, 3:07 a.m.)
> 
> 
> Review request for Build System and Plasma.
> 
> 
> Description
> -------
> 
> ksplashx: Adjust erroneous CMake calls.
> 
> Since its inception, ksplashx's CMakeLists.txt was using `include(FindFOO)'
> instead of `find_package(FOO)' to find libpng and libjpeg. While it has indeed
> worked so far, it is not the right way to find dependencies.
> 
> Adjust this by correctly looking for libpng and libjpeg via find_package(). The
> calls are in the top-level CMakeLists.txt since putting all find_package() and
> macro_optional_find_package() calls in the same file seems to appease some
> packagers.
> 
> While here, also add libjpeg's include directory to the compiler's include path
> and use a FindPNG variable that's not internal to do the same for libpng.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt ded5a2995b3d25c1c7c891afe93795fb0dfcfb87 
>   ksplash/ksplashx/CMakeLists.txt 65447939dc9ac17c69f1e94afe935b37c7dceba4 
> 
> Diff: http://git.reviewboard.kde.org/r/105468/diff/
> 
> 
> Testing
> -------
> 
> kde-workspace built fine both before and after this change
> 
> 
> Thanks,
> 
> Raphael Kubo da Costa
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20120723/3f3e50d4/attachment.html>


More information about the Kde-buildsystem mailing list