[PATCH] Fail early if libsm is not installed

Aurélien Gâteau agateau at kde.org
Wed Jan 8 13:52:36 UTC 2014


Le dimanche 29 décembre 2013 14:42:53 David Faure a écrit :
> 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.

Thanks, I just committed a simpler version of the patch, handling missing X11 
features the same way kidletime does.

Aurélien


More information about the Kde-frameworks-devel mailing list