Review Request 112839: Make add_library(...MODULE...) a proper replacement for kde4_add_plugin

Commit Hook null at kde.org
Fri Sep 20 14:14:54 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112839/
-----------------------------------------------------------

(Updated Sept. 20, 2013, 2:14 p.m.)


Status
------

This change has been marked as submitted.


Review request for Build System, KDE Frameworks, Alexander Neundorf, and Stephen Kelly.


Description
-------

One of the most important things kde4_add_plugin was doing was stripping MODULE targets from its prefix. This change makes it so those targets won't get the prefix by default.


Diffs
-----

  kde-modules/KDECMakeSettings.cmake 704e4ce 

Diff: http://git.reviewboard.kde.org/r/112839/diff/


Testing
-------

Builds.
Changed locally some kde4_add_plugin to add_library( MODULE ) and they didn't have a prefix.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130920/19f7ece0/attachment.html>


More information about the Kde-frameworks-devel mailing list