[Kde-pim] Review Request: Make kdepim-runtime configuration less verbose

[Aleix Pol Gonzalez]

> On Aug. 20, 2012, 4:37 p.m., Raphael Kubo da Costa wrote:
> > Doesn't the same change also apply to the other find_package() calls there?
> 
> Allen Winter wrote:
>     Yes, I'm also curious about why these 2 are so much more irritating than any other find_packages()
>     
>     Aleix, can you paste the annoying parts?  Maybe we can convince the libkolab folks to make their stuff less babbly.

Well, it's what happens when you look for a Config.cmake file that has to be installed.

Sorry for the delay, forgot about providing this :P.

CMake Warning at CMakeLists.txt:122 (find_package):
  By not providing "FindLibkolab.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "Libkolab",
  but CMake did not find one.

  Could not find a package configuration file provided by "Libkolab"
  (requested version 0.2) with any of the following names:

    LibkolabConfig.cmake
    libkolab-config.cmake

  Add the installation prefix of "Libkolab" to CMAKE_PREFIX_PATH or set
  "Libkolab_DIR" to a directory containing one of the above files.  If
  "Libkolab" provides a separate development package or SDK, be sure it has
  been installed.


CMake Warning at CMakeLists.txt:125 (find_package):
  By not providing "FindLibkolabxml.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "Libkolabxml", but CMake did not find one.

  Could not find a package configuration file provided by "Libkolabxml" with
  any of the following names:

    LibkolabxmlConfig.cmake
    libkolabxml-config.cmake

  Add the installation prefix of "Libkolabxml" to CMAKE_PREFIX_PATH or set
  "Libkolabxml_DIR" to a directory containing one of the above files.  If
  "Libkolabxml" provides a separate development package or SDK, be sure it
  has been installed.


- Aleix


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


On Aug. 20, 2012, 3:22 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106101/
> -----------------------------------------------------------
> 
> (Updated Aug. 20, 2012, 3:22 p.m.)
> 
> 
> Review request for Build System and KDEPIM.
> 
> 
> Description
> -------
> 
> in case the package is not required, don't print a ton of debug output but proceed quietly if it's not present.
> 
> the fact that it's not found is already notified by the log anyway.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt d91ed8a 
> 
> Diff: http://git.reviewboard.kde.org/r/106101/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/