----------------------------------------------------------- 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel