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