D17446: cmake macros: Port away from deprecated ECM var in kcoreaddons_add_plugin

Michael Pyne noreply at phabricator.kde.org
Sun Dec 9 03:57:50 GMT 2018


mpyne created this revision.
mpyne added reviewers: Frameworks, elvisangelaccio, apol, dfaure.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
mpyne requested review of this revision.

REVISION SUMMARY
  As reported in bug 401888 <https://bugs.kde.org/show_bug.cgi?id=401888>, the current `kcoreaddons_add_plugin` CMake macro causes deprecation warnings in extra-cmake-module's `KDEInstallDirs` since we use an old-style install path variable (`PLUGIN_INSTALL_DIR`).
  
  This should just be `KDE_INSTALL_PLUGINDIR` and so that's all my fix is, but since I'm not very familiar with the CMake stuff or the plugin loading code I wanted to get a review on it before I commit.

TEST PLAN
  Built and installed kcoreaddons successfully, and then built and installed kactivitymanagerd (which uses kcoreaddons_add_plugin pervasively) and verified that kactivitymanagerd still builds and installs and that the plugin install dir, including its install namespace, is properly updated.
  
  I also manually forced `KDE_INSTALL_DIRS_NO_DEPRECATED` to a true value in kactivitymanagerd as recommended in the bug, and still rebuilt/installed without deprecation warnings.

REPOSITORY
  R244 KCoreAddons

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

AFFECTED FILES
  KF5CoreAddonsMacros.cmake

To: mpyne, #frameworks, elvisangelaccio, apol, dfaure
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181209/92bddfa8/attachment.html>


More information about the Kde-frameworks-devel mailing list