D29051: Add ecm_generate_dbus_service_file
Friedrich W. H. Kossebau
noreply at phabricator.kde.org
Tue Apr 21 15:43:59 BST 2020
kossebau added a comment.
Some first quick comments, not yet looked at code itself.
Misses also link file from doc/.
INLINE COMMENTS
> ECMGenerateDBusServiceFile.cmake:6
> +# This module provides the ``ecm_generate_dbus_service_file`` function for
> +# generating a D-Bus service file.
> +#
generate and install
> ECMGenerateDBusServiceFile.cmake:22
> +#
> +# Since 5.70.0.
> +
Would be nice to have an example.
Also having each argument discussed in an own section makes getting the docs easier,
> ECMGenerateDBusServiceFile.cmake:90
> +
> + install(FILES ${_service_file} DESTINATION ${KDE_INSTALL_DBUSSERVICEDIR})
> +endfunction()
Creates a dependecy on KDEInstallDirs.
ECM/Modules are supposed to be usable in non-KDE-typical setups. So like other places in this subfolder this macro needs an argument to pass the installation folder.
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D29051
To: broulik, #frameworks, davidedmundson, kossebau, kfunk, habacker
Cc: kde-frameworks-devel, kde-buildsystem, LeGast00n, cblack, bencreasy, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20200421/fac045ba/attachment-0001.html>
More information about the Kde-buildsystem
mailing list