KF5 port: remove KF5::KDELibs4Support
ClosedPublic

Authored by chehrlic on Aug 19 2017, 6:05 PM.

Details

Summary

last patch (only 5, not 7 patches needed) to remove kdelibs4support from frameworks branch

Test Plan

this one removes the last pieces from CMakeLists.txt - only works after the other 4. Still compiles, but now kdelibs4support - free :)

Diff Detail

Repository
R423 Palapeli
Branch
d7420
Lint
No Linters Available
Unit
No Unit Test Coverage
chehrlic created this revision.Aug 19 2017, 6:05 PM
chehrlic updated this revision to Diff 18438.Aug 20 2017, 1:09 PM
  • Remove unneeded K_EXPORT_PLUGIN macros
Restricted Application added a reviewer: KDE Games. · View Herald TranscriptAug 20 2017, 1:09 PM
Restricted Application added a subscriber: KDE Games. · View Herald Transcript
chehrlic updated this revision to Diff 18439.Aug 20 2017, 1:46 PM

fighting with arc... sry

This one depends on D7416, D7417 and D7418

ltoscano set the repository for this revision to R423 Palapeli.Aug 22 2017, 8:24 PM
chehrlic updated this revision to Diff 18611.Aug 23 2017, 5:08 PM
  • Remove unused dependency to KF5::Completion and KF5::Service

You readded the K_EXPORT_PLUGIN macros, is it expected?

chehrlic updated this revision to Diff 18641.Aug 24 2017, 4:48 AM
  • Remove unneeded K_EXPORT_PLUGIN macros
chehrlic updated this revision to Diff 18649.Aug 24 2017, 7:02 AM
  • Remove useless includes to KPluginLoader
ltoscano accepted this revision.Aug 24 2017, 7:58 PM
This revision is now accepted and ready to land.Aug 24 2017, 7:58 PM
chehrlic closed this revision.Aug 25 2017, 5:48 PM