D15573: replace custom backtracing in SlaveBase with KCrash
Harald Sitter
noreply at phabricator.kde.org
Mon Sep 17 13:26:22 BST 2018
sitter created this revision.
sitter added a reviewer: dfaure.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
sitter requested review of this revision.
REVISION SUMMARY
as discussed on mailing list.
KCrash originally was automatically initialized, but now needs to be
explicitly linked to work.
additionally the custom backtrace() in the SlaveBase is only marginally
useful and largely replaced by improvements to KCrash (such as core pattern
support on linux to allow raising crashes to a global core collector such
as systemd's coredumpd).
If someone wants to resurrect the custom backtrace() feature it needs to
be moved to KCrash and actually made signal-safe (explicitly load libgcc;
see backtrace manpage).
CHANGELOG: KIO slave crashes are now handled by KCrash instead of subpar custom code
TEST PLAN
- q_assert(false) in a slave now brings up drkonqi
REPOSITORY
R241 KIO
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D15573
AFFECTED FILES
CMakeLists.txt
src/core/CMakeLists.txt
src/core/ConfigureChecks.cmake
src/core/config-kiocore.h.cmake
src/core/slavebase.cpp
To: sitter, dfaure
Cc: kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180917/6b0252d1/attachment.html>
More information about the Kde-frameworks-devel
mailing list