D28609: Add macro EXPORT_KONTACT_PLUGIN_WITH_JSON

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Mon Apr 6 21:47:16 BST 2020


kossebau added a comment.


  In D28609#643225 <https://phabricator.kde.org/D28609#643225>, @dfaure wrote:
  
  > Use ECMSetupQtPluginMacroNames.
  >
  > Not sure I see the point in writing 6+1 lines in order to generate 3
  >  (or 3+1 vs 3 with less wrapping), but OK :)
  
  
  Yes, it makes more sense in projects which use the macro also in the same project.  Here I proposed it more for the sake of consistency, and promoting its existance, so others just copying code would learn about it.

INLINE COMMENTS

> CMakeLists.txt:43
> +    JSON_ARG3
> +       EXPORT_MYPLUGIN_WITH_JSON
> +   CONFIG_CODE_VARIABLE

You wanted "EXPORT_KONTACT_PLUGIN_WITH_JSON" here :)

REPOSITORY
  R85 PIM: Kontact Interface

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D28609

To: dfaure, winterz, mlaurent
Cc: kossebau, kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20200406/57ee2db7/attachment-0001.html>


More information about the kde-pim mailing list