Review Request 117652: Rewrite FindLibGcrypt.cmake to not use gcrypt-config

Commit Hook null at kde.org
Mon Apr 21 14:34:42 UTC 2014


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


This review has been submitted with commit 7035a9f45ee74481fad114a26a964c356aec732a by Nicolás Alvarez to branch master.

- Commit Hook


On April 21, 2014, 12:06 a.m., Nicolás Alvarez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117652/
> -----------------------------------------------------------
> 
> (Updated April 21, 2014, 12:06 a.m.)
> 
> 
> Review request for Build System and KDE Frameworks.
> 
> 
> Repository: kwallet
> 
> 
> Description
> -------
> 
> Using the gcrypt-config shell script won't work on Windows.
> 
> I wrote a new find module from scratch. It now searches for the library and include directory the usual CMake way (find_library and find_path). It supports version checking by extracting the version number from the gcrypt.h header.
> 
> By the way, lxr shows there are four different copies of FindLibGcrypt.cmake in different KDE projects and they all have one flaw or another. I think none uses FindPackageHandleStandardArgs, for example. Maybe we should consider putting this one into ECM for kwallet and the others to share.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 9aad9c0 
>   cmake/FindLibGcrypt.cmake 45c0d5d 
>   src/runtime/kwalletd/backend/CMakeLists.txt 4c36767 
> 
> Diff: https://git.reviewboard.kde.org/r/117652/diff/
> 
> 
> Testing
> -------
> 
> Tested on Windows with MSVC2010, cmake 2.8.12, finding emerge-built libgcrypt. Also tested on Debian Linux, cmake 2.8.12.1, finding the distro's libgcrypt.
> 
> 
> Thanks,
> 
> Nicolás Alvarez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140421/906e662a/attachment.html>


More information about the Kde-frameworks-devel mailing list