Review of File to be Added to kdelibs/cmake/modules
paulo.romulo at kdemail.net
Tue Aug 31 03:38:18 CEST 2010
2010/8/27 Maciej Mrozowski <reavertm at gmail.com>
> 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
> 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."
> 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
> 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