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

Vadim Zhukov persgray at gmail.com
Wed Aug 6 16:23:50 BST 2014


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

(Updated Aug. 6, 2014, 3:23 p.m.)


Status
------

This change has been marked as submitted.


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