Provide debugger name and pid when registering a debugger to DrKonqi
ClosedPublic

Authored by croick on Jan 25 2019, 9:36 AM.

Details

Summary

Adapts the DBus call to the change in DrKonqi commit 9f3cfcff960c.

Test Plan

Use the debug button in DrKonqi to attach a crashed application
to a debugger in KDevelop.

Diff Detail

Repository
R32 KDevelop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
croick created this revision.Jan 25 2019, 9:36 AM
Restricted Application added a project: KDevelop. · View Herald TranscriptJan 25 2019, 9:36 AM
Restricted Application added a subscriber: kdevelop-devel. · View Herald Transcript
croick requested review of this revision.Jan 25 2019, 9:36 AM

Do you think we should check for the version of Plasma to make a distinction between DrKonqi versions?
I think not, the Debug feature is not (yet) very popular, but I would like another opinion.

mwolff accepted this revision.Jan 25 2019, 2:25 PM
mwolff added a subscriber: mwolff.

I think it's fine to commit this as-is

This revision is now accepted and ready to land.Jan 25 2019, 2:25 PM
This revision was automatically updated to reflect the committed changes.