[Digikam-devel] Review Request 103701: Install LibKexiv2Config.cmake

[Alexander Neundorf]

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


You should include the target file only if the target does not already exist, something like
if(NOT TARGET Libkexiv2::kexiv2)
  include(...cmake)
endif()



I would recommend not to disable the set_and_check() macro, and use it for the include dir instead of simply set().
set_and_check() will set the variable and check that the directory it points to actually exists.
This is good, because it ensures that the config.cmake file provides information which is actually consistent with the state in disk.


- Alexander Neundorf


On May 5, 2013, 8:02 p.m., Yury Kudryashov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103701/
> -----------------------------------------------------------
> 
> (Updated May 5, 2013, 8:02 p.m.)
> 
> 
> Review request for Build System and Digikam.
> 
> 
> Description
> -------
> 
> Now
> find_package(libkexiv2 CONFIG)
> finds LibKexiv2, removing need for large FindKexiv2.cmake script.
> 
> I've decided to install libkexiv2-config.cmake, not Kexiv2Config.cmake to avoid conflicts with FindKexiv2.cmake from kdelibs.
> 
> Also, I'm not sure about libkexiv2-config-version.cmake. I'm ready to adjust the default policy if someone will tell me about correct algorithm (SONAME? Same major version? Same major/minor? something else?).
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 8c89503ffb2de66be682af3b262756519db3af49 
>   libkexiv2-config.cmake.in PRE-CREATION 
>   libkexiv2/CMakeLists.txt c706d820f54b412f45131a14276b13bfc07255a1 
> 
> Diff: http://git.reviewboard.kde.org/r/103701/diff/
> 
> 
> Testing
> -------
> 
> $ cmake .. -DCMAKE_INSTALL_PREFIX=$PWD/../install; cd ..;
> $ cmake --find-package -DCMAKE_PREFIX_PATH=$PWD/../install -DNAME=libkexiv2 -DCOMPILER_ID=GNU -DLANGUAGE=CXX -DMODE=EXIST
> $ cmake --find-package -DCMAKE_PREFIX_PATH=$PWD/../install -DNAME=libkexiv2 -DCOMPILER_ID=GNU -DLANGUAGE=CXX -DMODE=COMPILE
> $ cmake --find-package -DCMAKE_PREFIX_PATH=$PWD/../install -DNAME=libkexiv2 -DCOMPILER_ID=GNU -DLANGUAGE=CXX -DMODE=LINK
> 
> 
> Thanks,
> 
> Yury Kudryashov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20130506/4adecdfe/attachment.html>