Review Request 113670: Only link to Qt5::X11Extras and ${X11_LIBRARIES} if X11 was found

[Alexander Richardson]

> On Nov. 10, 2013, 9:50 a.m., David Faure wrote:
> > tier4/kde4support/src/CMakeLists.txt, line 293
> > <http://git.reviewboard.kde.org/r/113670/diff/3/?file=212165#file212165line293>
> >
> >     Why was this line removed?
> >     
> >     Everything else looks fine to me, this remove seems unrelated though.

I moved it into the if(X11_FOUND) block. Probably doesn't make any difference since it will be empty if X11 is not found. But I think it is nicer that way.


- Alexander


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


On Nov. 8, 2013, 10:04 p.m., Alexander Richardson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113670/
> -----------------------------------------------------------
> 
> (Updated Nov. 8, 2013, 10:04 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> 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
> -----
> 
>   tier4/kde4support/src/CMakeLists.txt cbfac3e 
>   tier4/kde4support/src/kdeui/kxerrorhandler.h babf931 
>   tier4/kde4support/src/kdeui/kxerrorhandler.cpp 3f4765d 
> 
> Diff: http://git.reviewboard.kde.org/r/113670/diff/
> 
> 
> Testing
> -------
> 
> 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/20131110/e2032daa/attachment.html>