Review Request 114701: Print a warning if kimagecache.h is used without KF5::CoreAddons
Alex Merry
kde at randomguy3.me.uk
Sat Dec 28 13:03:07 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114701/
-----------------------------------------------------------
Review request for KDE Frameworks.
Repository: kguiaddons
Description
-------
Print a warning if kimagecache.h is used without KF5::CoreAddons
If an application links against KF5::GuiAddons but not KF5::CoreAddons,
they cannot use KImageCache. So we use defines (set by the exported
targets) to check whether the current target has been linked against
KF5::CoreAddons, and print a warning if not (to explain the expected
"kshareddatacache.h not found" error).
Diffs
-----
src/CMakeLists.txt d6ee3d9987319f9a3d1f8b2f66e00d2ec4bda1e5
src/util/kimagecache.h 007891c757de9bedc5cf0c3b734e250004727ec4
Diff: https://git.reviewboard.kde.org/r/114701/diff/
Testing
-------
plasma-framework (which uses KImageCache) still builds fine. If an older version of KCoreAddons (without the KCOREADDONS_LIB define) is installed, a warning is printed when kimagecache.h is included. If the patch in review 114700 is applied to KCoreAddons, the warning is not printed.
Thanks,
Alex Merry
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131228/ec37b853/attachment.html>
More information about the Kde-frameworks-devel
mailing list