Review Request 118026: Fix KF5_ADD_KDEINIT_EXECUTABLE on Windows

Patrick Spendrin ps_ml at gmx.de
Wed May 7 08:40:22 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118026/#review57477
-----------------------------------------------------------

Ship it!


looks good, I don't remember what weird reason required us to use the current setup. If it becomes a problem, we have to modify it anyway.

- Patrick Spendrin


On May 6, 2014, 8:29 p.m., Alexander Richardson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118026/
> -----------------------------------------------------------
> 
> (Updated May 6, 2014, 8:29 p.m.)
> 
> 
> Review request for KDE Frameworks and kdewin.
> 
> 
> Repository: kinit
> 
> 
> Description
> -------
> 
> Fix KF5_ADD_KDEINIT_EXECUTABLE on Windows
> 
> No longer create an empty dummy library, instead create a static library
> with all the sources and link the exectuable to that.
> 
> This fixes the build of Kate on Windows since some unit tests use
> target_link_libraries(test kdeinit_kate) and since that library was empty
> it would not contain the required symbols
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 9e34058ab612732adba50331db624d2c307da0dd 
>   KF5InitMacros.cmake 1768bc96573ca0fabf6bf620e4ff0a3f5f3646a5 
>   kde5init_win32lib_dummy.cpp.in d727297578834ba2f2c3ee8f086c355244f116de 
> 
> Diff: https://git.reviewboard.kde.org/r/118026/diff/
> 
> 
> Testing
> -------
> 
> Kate tests link now, didn't before
> 
> 
> Thanks,
> 
> Alexander Richardson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-windows/attachments/20140507/ca13e8d0/attachment-0001.html>


More information about the Kde-windows mailing list