mpyne accepted this revision.
mpyne added a comment.
This revision is now accepted and ready to land.


  The change looks good and builds in various combinations of CMake options 
(after I installed the changeset from D23789 
<https://phabricator.kde.org/D23789>). I encountered that excluding deprecated 
components from the build requires a version of x.y.z. (not just x.y) but other 
than that things were straightforward from a developer's perspective.
  
  Once D23789 <https://phabricator.kde.org/D23789> is in we can commit to 
kcoreaddons.

REPOSITORY
  R244 KCoreAddons

BRANCH
  useECMGenerateExportHeader

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

To: kossebau, #frameworks, mpyne
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

Reply via email to