Review of File to be Added to kdelibs/cmake/modules

Paulo Rômulo paulo.romulo at
Tue Aug 31 03:38:18 CEST 2010

2010/8/27 Maciej Mrozowski <reavertm at>

> On Thursday 26 of August 2010 20:59:21 Paulo Rômulo wrote:
> > > > I'm intending to add the attached file to kdelibs/cmake/modules in
> > > > order to conditionally build the Solid UPnP backend.
> Using find_package(xxx) alone introduces automagic dependency (dependency
> that
> is determined at build time and cannot be overriden by the means of cmake
> configure options). Also commit message
>  "Building Solid UPnP backend by default now. HUPnP library required."
> doesn't
> exactly say hupnp library is an optional dependency for kdelibs so it's not
> clear what your intentions really are.
> Sorry for that. And, yes, it is an optional dependency.

> So, if HUPnp backend is indeed meant to be optional:
> - replace find_package with macro_optional_find_package (adds WITH_HUpnp
> CMake
> variable to switch on/off this dependency)
> - add macro_log_feature just below macro_optional_find_package(HUpnp) - in
> such case message(STATUS "Building Solid UPnP backend." ) can be dropped
> (macro_log_feature will take the role of informing about that)
> Done. Please, tell me if it's fine now.


-------------- next part --------------
An HTML attachment was scrubbed...

More information about the Kde-buildsystem mailing list