Review Request 112874: Allow disabling of QCA plugins during build.

[Ivan Romanov]

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


Fail.

BUILD_PLUGIN(S). You missed 'S'. BUILD_PLUGINS is a list not an option, use set(BUILD_PLUGINS "all" CACHE STRING "..."). Correct INSTALL, BUILD_PLUGINS must be 'all' by default. 

Also cmake must be fail if BUILD_PLUGINS it not 'all' and some plugin can't be built.

- Ivan Romanov


On Oct. 13, 2013, 5:32 p.m., Uwe L. Korn wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112874/
> -----------------------------------------------------------
> 
> (Updated Oct. 13, 2013, 5:32 p.m.)
> 
> 
> Review request for KDE Frameworks and Ivan Romanov.
> 
> 
> Repository: qca
> 
> 
> Description
> -------
> 
> Add the ability to disable plugins, e.g. do not build the openssl plugins although OpenSSL was found in the system. This is a requirement that arose during packaging latest head for Gentoo.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt bdb77b4 
>   INSTALL cf847a0 
>   plugins/CMakeLists.txt 6e8a7a2 
>   plugins/qca-botan/CMakeLists.txt 1b9607c 
>   plugins/qca-cyrus-sasl/CMakeLists.txt b36cbcf 
>   plugins/qca-gcrypt/CMakeLists.txt f5c73bd 
>   plugins/qca-gnupg/CMakeLists.txt 7f46abf 
>   plugins/qca-nss/CMakeLists.txt 805567d 
>   plugins/qca-ossl/CMakeLists.txt 6b5c69c 
>   plugins/qca-pkcs11/CMakeLists.txt b220926 
>   plugins/qca-wincrypto/CMakeLists.txt c0ab378 
> 
> Diff: http://git.reviewboard.kde.org/r/112874/diff/
> 
> 
> Testing
> -------
> 
> Compiled, installed and used on Gentoo (with Qt5).
> 
> 
> Thanks,
> 
> Uwe L. Korn
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131013/8803a43c/attachment.html>