Differential D20081

Add readme explaining how kwallet-pam works
ClosedPublic
Actions

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

Details

Reviewers
apol
Group Reviewers
Plasma
Commits
R107:b0ac7bbd6adf: Add readme explaining how kwallet-pam works
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
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 10198
Build 10216: arc lint + arc unit
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
3

plain<space>text

9

QApp -> QApplication

13

kwalletd

Also what's "the env"?

davidedmundson updated this revision to Diff 55320.Apr 2 2019, 10:37 PM
davidedmundson marked 3 inline comments as done.

update

davidedmundson added a comment.Apr 2 2019, 10:37 PM

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
Closed by commit R107:b0ac7bbd6adf: Add readme explaining how kwallet-pam works (authored by davidedmundson). · Explain WhyApr 3 2019, 10:56 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
AMREADME.txt (16 lines)

Diff 54940

View Options

README.txt

Loading...