Review Request 113670: Link kde4support privately to QtX11Extras, QtSvg and QtTest

[Alexander Richardson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113670/
-----------------------------------------------------------

(Updated Nov. 7, 2013, 9:33 p.m.)


Review request for KDE Frameworks.


Summary (updated)
-----------------

Link kde4support privately to QtX11Extras, QtSvg and QtTest


Repository: kdelibs


Description (updated)
-------

Link kde4support privately to QtX11Extras, QtSvg and QtTest

Otherwise every user of the target KF5::KDE4Support will also have
Qt5::X11Extras Qt5::Svg Qt5::Test linked.

This can cause linker errors like "ld: cannot find -lQt5::Test"

REVIEW: 113670


Diffs (updated)
-----

  tier4/kde4support/src/kdeui/kxerrorhandler.cpp 3f4765d03559c99833c573fe0e0072bc9973b973 
  tier4/kde4support/src/kdeui/kxerrorhandler.h babf931e8a750d405885f02443f4910356511429 
  tier4/kde4support/src/CMakeLists.txt cbfac3e9e4363cb66d0941c9048d8f501311084c 

Diff: http://git.reviewboard.kde.org/r/113670/diff/


Testing (updated)
-------

I previously got the following error compiling okteta, now it works:
/usr/lib64/gcc/x86_64-suse-linux/4.7/../../../../x86_64-suse-linux/bin/ld: cannot find -lQt5::X11Extras
/usr/lib64/gcc/x86_64-suse-linux/4.7/../../../../x86_64-suse-linux/bin/ld: cannot find -lQt5::Svg

QtX11Info and QtSvg are not used in any installed headers, so IMO this should be fine
qtest_kde.h does actually include QtTest headers, but only uses these types inside macros. And I guess any user of that header already also links to QtTest


Thanks,

Alexander Richardson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131107/797fd5d0/attachment.html>