Please review: cmake check for Cagibi
Christophe Giboudeaux
cgiboudeaux at gmail.com
Tue Aug 24 21:45:27 CEST 2010
Hi,
Le 24/08/2010 21:31, Friedrich W. H. Kossebau a écrit :
> Mardi, le 24 août 2010, à 21:23, vous avez écrit:
>> > On Tuesday 24 August 2010, Friedrich W. H. Kossebau wrote:
>>> > > Mardi, le 24 août 2010, à 20:40, vous avez écrit:
>>> > > So okay to commit the attached patch, both to trunk and 4.5 branch?
>> >
>> > No, no.
>> >
>> > Index: kdebase/runtime/cmake/modules/CMakeLists.txt
>> > ===================================================================
>> > --- kdebase/runtime/cmake/modules/CMakeLists.txt (Revision 1157717)
>> > +++ kdebase/runtime/cmake/modules/CMakeLists.txt (Arbeitskopie)
>> > @@ -2,6 +2,7 @@
>> >
>> > # don't glob for the files, we don't want to install e.g.
>> > FindLibKNotification-1.cmake
>> > set(cmakeFiles FindCLucene.cmake
>> > + FindCagibiDaemon.cmake
>> > FindSLP.cmake)
>> >
>> > set(module_install_dir ${DATA_INSTALL_DIR}/cmake/modules )
>> >
>> >
>> > Do you *really* want to install that file ?
>> > I think that shouldn't be necessary.
> Alright. What about this attached version? :)
>
[cut]
> )
> +macro_optional_find_package(CagibiDaemon)
> +macro_log_feature( CAGIBI_DAEMON_FOUND
> + "Cagibi daemon" "SSDP (Simple Service Discovery Protocol, used by UPnP) cache/proxy daemon (RUNTIME DEPENDENCY)" "http://websvn.kde.org/trunk/kdesupport/cagibi/" FALSE
> + "0.1.1" "Provides UPnP device/service listing in the network:/ kio-slave."
> +)
>
Sorry but I still think it's a bad idea. macro_log_feature() should only
report missing build dependencies or in the future we'll have all kind
of not-so-useful messages about runtime deps in different modules.
If Cagibi has no usefulness at build time, look for it at runtime only.
a kFatal/kWarning was suggested last week.
a README.packagers would be enough (that's what we have in eg. kdepim).
Christophe.
More information about the Kde-buildsystem
mailing list