KIO: fix compilation with KF_DISABLE_DEPRECATED_BEFORE_AND_AT=0x054200
ClosedPublic

Authored by dfaure on Jan 11 2020, 8:04 PM.

Details

Summary

KPixmapProvider is not available anymore once this is set.

Test Plan

Builds

Diff Detail

Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 20974
Build 20992: arc lint + arc unit
dfaure requested review of this revision.Jan 11 2020, 8:04 PM
dfaure created this revision.
nicolasfella accepted this revision.Jan 11 2020, 8:05 PM
This revision is now accepted and ready to land.Jan 11 2020, 8:05 PM
dfaure closed this revision.Jan 12 2020, 9:56 AM
kossebau added a subscriber: kossebau.EditedJan 13 2020, 8:08 AM

By setting -DKF_DISABLE_DEPRECATED_BEFORE_AND_AT=0x054200 for the build of the lib, you effectively disable KCOMPLETION_ENABLE_DEPRECATED_SINCE(5, 66)​, and thus hide this class from any builds. Which makes kdelibs4support fail e.g., cmp. https://build.kde.org/job/Frameworks/job/kdelibs4support/job/kf5-qt5%20SUSEQt5.12/86/

You need to also add ​add_definitions(-DKCOMPLETION_DISABLE_DEPRECATED_BEFORE_AND_AT=0x054100) to still see KPixmapProvider all the time when building this lib, as this override the group settings from the KF variant.

Will do a quick hot fix then to unbreak builds, proper solution can be discussed later (only had my first coffee right now).

Thanks. I was about to push the exact same hotfix at the same moment (well, one minute later) :-)