Review Request 118026: Fix KF5_ADD_KDEINIT_EXECUTABLE on Windows
    Alexander Richardson 
    arichardson.kde at googlemail.com
       
    Wed May  7 14:07:52 UTC 2014
    
    
  
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118026/
-----------------------------------------------------------
(Updated May 7, 2014, 2:07 p.m.)
Status
------
This change has been marked as submitted.
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-frameworks-devel/attachments/20140507/76bca1f7/attachment.html>
    
    
More information about the Kde-frameworks-devel
mailing list