D24979: KRunner: port away from deprecated KF5 API

[Friedrich W. H. Kossebau]

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


  Untested, but looks okay, besides the unneeded #f in the sources.

INLINE COMMENTS

> abstractrunner.cpp:327
>  
> +#if PLASMA_ENABLE_DEPRECATED_SINCE(5, 28) // Plasma::Package is defined with this condition
>  Package AbstractRunner::package() const

This one should be removed.

We have defined

  add_definitions(-DKF_DISABLE_DEPRECATED_BEFORE_AND_AT=0x053f00)
  add_definitions(-DPLASMA_DISABLE_DEPRECATED_BEFORE_AND_AT=0x050500) # needed because we use Plasma::Package in the API

with this in the build of KRunner itselt PLASMA_ENABLE_DEPRECATED_SINCE(5, 28) is always true, and people building KRunner do not have the option to change this.

This condition is only interesting in the header currently, as this is what then is used by 3rd-party building against KRunner and being allowed to set different visibility limits.

REPOSITORY
  R308 KRunner

BRANCH
  D24979

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

To: dfaure, kossebau, mart, apol
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191103/7ada3280/attachment-0001.html>