Differential D17119

actually initialize kcrash properly
ClosedPublic
Actions

Authored by sitter on Nov 23 2018, 9:53 AM.

Details

Reviewers
ivan
davidedmundson
Group Reviewers
Plasma
Commits
R161:699dbef8f577: actually initialize kcrash properly
Summary

to successfully use kcrash when linking with as-needed (which is a default
flag on many linux distros) one also needs to call KCrash::initialize.

without this kcrash may not be available for most users.

https://markmail.org/thread/zv5pheijaze72bzs

(kcrash is already in the link target list)

Test Plan

builds; links correctly

Diff Detail

Repository
R161 KActivity Manager Service
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sitter created this revision.Nov 23 2018, 9:53 AM
Restricted Application added a project: Plasma. · View Herald TranscriptNov 23 2018, 9:53 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
sitter requested review of this revision.Nov 23 2018, 9:53 AM
davidedmundson accepted this revision.Nov 23 2018, 10:33 AM
This revision is now accepted and ready to land.Nov 23 2018, 10:33 AM
Closed by commit R161:699dbef8f577: actually initialize kcrash properly (authored by sitter). · Explain WhyNov 23 2018, 10:40 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/service/Application.cpp (4 lines)

Diff 46071

View Options

src/service/Application.cpp

Loading...