Review Request 112495: Move kcrash to tier2

Martin Klapetek martin.klapetek at gmail.com
Wed Sep 4 11:03:25 UTC 2013



> On Sept. 4, 2013, 10:47 a.m., Stephen Kelly wrote:
> > staging/kcrash/CMakeLists.txt, line 12
> > <http://git.reviewboard.kde.org/r/112495/diff/2/?file=186875#file186875line12>
> >
> >     Why was this needed? Widgets already finds Qt5Gui.

Isn't it the other way around (Gui finds Widgets)? It was added because it links to Gui because it needs qwindowdefs.h, which is in Gui.


- Martin


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


On Sept. 4, 2013, 10:01 a.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112495/
> -----------------------------------------------------------
> 
> (Updated Sept. 4, 2013, 10:01 a.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Description
> -------
> 
> Removes the remove_definitions(..) from cmake, no code changes needed. KCrash however has no tests, I'm not sure how to test it properly (if someone can give me ideas, I'll try writing such tests).
> 
> 
> Diffs
> -----
> 
>   staging/CMakeLists.txt 3e4ec3a 
>   staging/kcrash/CMakeLists.txt ce926ec 
>   staging/kcrash/KCrashConfig.cmake.in  
>   staging/kcrash/src/CMakeLists.txt 5dab9be 
>   staging/kcrash/src/config-kcrash.h.cmake  
>   staging/kcrash/src/config-strlcpy.h.cmake  
>   staging/kcrash/src/kcrash.h  
>   staging/kcrash/src/kcrash.cpp  
>   staging/kcrash/src/strlcpy-fake.c  
>   tier2/CMakeLists.txt bd5ea8c 
> 
> Diff: http://git.reviewboard.kde.org/r/112495/diff/
> 
> 
> Testing
> -------
> 
> Builds ok.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

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


More information about the Kde-frameworks-devel mailing list