Re: [Freeipa-devel] [PATCH] add man page for ipa-kra-install

2014-08-26 Thread Petr Viktorin
On 08/25/2014 06:37 PM, Ade Lee wrote: New patch attached. If OK, please commit for me. Thanks, Ade I missed the argument list, where you have a deprecated option, and you list -U for both --unattended and --uninstall. Here's an updated patch, I can push if it looks OK to you. -- PetrĀ³

Re: [Freeipa-devel] [PATCH] add man page for ipa-kra-install

2014-08-26 Thread Ade Lee
Looks good to me. Thanks. Ade On Tue, 2014-08-26 at 14:13 +0200, Petr Viktorin wrote: On 08/25/2014 06:37 PM, Ade Lee wrote: New patch attached. If OK, please commit for me. Thanks, Ade I missed the argument list, where you have a deprecated option, and you list -U for both

Re: [Freeipa-devel] [PATCH] add man page for ipa-kra-install

2014-08-26 Thread Petr Viktorin
On 08/26/2014 02:47 PM, Ade Lee wrote: Looks good to me. Thanks. Ade On Tue, 2014-08-26 at 14:13 +0200, Petr Viktorin wrote: On 08/25/2014 06:37 PM, Ade Lee wrote: New patch attached. If OK, please commit for me. Thanks, Ade I missed the argument list, where you have a deprecated option,

Re: [Freeipa-devel] [PATCH] add man page for ipa-kra-install

2014-08-25 Thread Petr Viktorin
On 08/24/2014 06:28 PM, Ade Lee wrote: Added man pages for ipa-kra-install. And its not even Tuesday yet :) Please review, Ade If I was new to this, I think I'd be quite lost. I think the man page should briefly explain what KRA is -- just a sentence would be fine. At the very least

Re: [Freeipa-devel] [PATCH] add man page for ipa-kra-install

2014-08-25 Thread Ade Lee
What if I add the following first paragraph? The KRA (Key Recovery Authority) is a component used to securely store secrets such as passwords, symmetric keys and private asymmetric keys. It is used as the back-end repository for the IPA Password Vault. Ade On Mon, 2014-08-25 at 10:28 +0200,

Re: [Freeipa-devel] [PATCH] add man page for ipa-kra-install

2014-08-25 Thread Petr Viktorin
On 08/25/2014 06:17 PM, Ade Lee wrote: What if I add the following first paragraph? The KRA (Key Recovery Authority) is a component used to securely store secrets such as passwords, symmetric keys and private asymmetric keys. It is used as the back-end repository for the IPA Password Vault.

Re: [Freeipa-devel] [PATCH] add man page for ipa-kra-install

2014-08-25 Thread Ade Lee
New patch attached. If OK, please commit for me. Thanks, Ade On Mon, 2014-08-25 at 18:25 +0200, Petr Viktorin wrote: On 08/25/2014 06:17 PM, Ade Lee wrote: What if I add the following first paragraph? The KRA (Key Recovery Authority) is a component used to securely store secrets such as