[PATCH] Fail early if libsm is not installed

[David Faure]

On Thursday 19 December 2013 18:10:11 Aurélien Gâteau wrote:
> Hi,
> 
> Attached patch causes cmake to fail early if libsm is not installed, rather
> than waiting until we try to build code using it.
> 
> This patch also removes buggy calls to set_package_properties(). I found
> out calling set_package_properties() with something which is not a
> package as the first argument is currently silently ignored. Note that
> this means the TYPE argument is ignored as well, so marking something as
> REQUIRED has no effect.
> 
> As a result I put together a poor-man feature_summary() implementation
> at the end of the CMakeLists.txt for X11 components. It's a bit overkill
> for the needs of kde4support, but can be duplicated or factorized
> later.

I can't comment on whether there's a better way to do this, but anyway, the 
patch improves things so it should go in IMHO.

-- 
David Faure, faure at kde.org, http://www.davidfaure.fr
Working on KDE, in particular KDE Frameworks 5