[Kde-pim] Review Request 119630: Use FindBacktrace.cmake instead of rolling your own

Dan Vrátil dvratil at redhat.com
Wed Aug 6 16:20:13 BST 2014


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

Ship it!


- Dan Vrátil


On Aug. 6, 2014, 4:21 p.m., Vadim Zhukov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119630/
> -----------------------------------------------------------
> 
> (Updated Aug. 6, 2014, 4:21 p.m.)
> 
> 
> Review request for Akonadi and Build System.
> 
> 
> Repository: akonadi
> 
> 
> Description
> -------
> 
> Akonadi uses backtrace() facility, which could be defined in different headers and actually live in different libraries, depending on OS. The CMake distribution has FindBacktrace.cmake module as of 3.0.
> 
> In case it's not desirable to depend on CMake 3.x, and given that CMake modules are released under BSD-style license, the FindBacktrace.cmake from CMake distribution could be used. This CMake module could be downloaded here: https://github.com/Kitware/CMake/raw/master/Modules/FindBacktrace.cmake
> 
> This patch removes more lines than adds, so it can't be bad. ;) And it fixes official Akonadi package build on OpenBSD.
> 
> Please note that I don't have commit access to KDE repos. Thus, if this patch will get accepted, it should be committed by someone else. Thanks.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 5bbe59f 
>   cmake/modules/FindBacktrace.cmake PRE-CREATION 
>   config-akonadi.h.cmake 5e4c4b9 
>   server/tests/unittest/imapstreamparsertest.cpp 738147b 
>   shared/akcrash.cpp 87662fb 
> 
> Diff: https://git.reviewboard.kde.org/r/119630/diff/
> 
> 
> Testing
> -------
> 
> Akonadi 1.12.1 and "master" ones, OpenBSD/i386 5.6-BETA, KDE 4.13.3
> 
> 
> Thanks,
> 
> Vadim Zhukov
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/


More information about the kde-pim mailing list