Differential D11950

Remove traces of polkit-kde-authorization tool
ClosedPublic
Actions

Authored by broulik on Apr 5 2018, 10:33 AM.

Details

Reviewers
davidedmundson
fvogt
Commits
R121:3791346b2843: Remove traces of polkit-kde-authorization tool
Summary

It's been unmaintained for almost a decade now and even the code asks "what's this?".
Instead, show the action ID in a label that allows text selection so it can be copied to clipboard if needed.

Test Plan

Diff Detail

Repository
R121 Policykit (Polkit) KDE Agent
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Apr 5 2018, 10:33 AM
Restricted Application added a project: Plasma. · View Herald TranscriptApr 5 2018, 10:33 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Apr 5 2018, 10:33 AM
davidedmundson accepted this revision.Apr 5 2018, 10:44 AM
This revision is now accepted and ready to land.Apr 5 2018, 10:44 AM
fvogt accepted this revision.Apr 5 2018, 10:58 AM

I don't even have that binary installed and no package contains it here - probably some legacy KDE4 stuff.

Closed by commit R121:3791346b2843: Remove traces of polkit-kde-authorization tool (authored by broulik). · Explain WhyApr 5 2018, 12:19 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
MAuthDialog.h (1 line)
MAuthDialog.cpp (11 lines)
Mauthdetails.ui (88 lines)

Diff 31389

View Options

AuthDialog.h

Loading...
View Options

AuthDialog.cpp

Loading...
View Options

authdetails.ui

Loading...