Add readme explaining how kwallet-pam works
ClosedPublic

Authored by davidedmundson on Mar 27 2019, 4:25 PM.

Details

Summary

I wasted too long before starting some hacking, I want
to make it easier for the next person

Test Plan

N/A

Diff Detail

Repository
R107 KWallet PAM Integration
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Mar 27 2019, 4:25 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMar 27 2019, 4:25 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Mar 27 2019, 4:25 PM
apol added a subscriber: apol.Mar 27 2019, 11:38 PM

+1 for documenting and the text.

Decorating with some URLs to pam documentation would be nice.

README.txt
4

plain<space>text

10

QApp -> QApplication

14

kwalletd

Also what's "the env"?

davidedmundson marked 3 inline comments as done.

update

addressed comments

apol accepted this revision.Apr 3 2019, 12:01 AM

LGTM

This revision is now accepted and ready to land.Apr 3 2019, 12:01 AM
This revision was automatically updated to reflect the committed changes.