Differential D17018

actually initialize kcrash properly
ClosedPublic
Actions

Authored by sitter on Nov 19 2018, 3:45 PM.

Details

Reviewers
dfaure
Commits
R226:e86622440e58: Merge remote-tracking branch 'origin/Applications/18.12'
R226:91aa44e615d6: 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

Test Plan

builds; links as expected

Diff Detail

Repository
R226 Konqueror
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
sitter requested review of this revision.Nov 19 2018, 3:45 PM
sitter created this revision.
dfaure accepted this revision.Nov 19 2018, 9:30 PM
This revision is now accepted and ready to land.Nov 19 2018, 9:30 PM
Closed by commit R226:91aa44e615d6: actually initialize kcrash properly (authored by sitter). · Explain WhyNov 20 2018, 10:16 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/konqmain.cpp (5 lines)

Diff 45872

View Options

src/konqmain.cpp

Loading...