Re: [Pki-devel] [PATCH] 623 Added pki-audit man page.

On 7/15/2015 5:13 PM, Endi Sukma Dewata wrote: > Thanks for the comments. Please take a look at the updated patch. > > On 7/14/2015 12:17 PM, Matthew Harmsen wrote: >> This was a little confusing as the ticket is for a kra-audit man page. >> I think I understand a bit better after reading comment #4, > > Yes, the command is removed from all subsystems except TPS because of > the database upgrade requirement. > >> however, I >> still have the following questions/comments: >> >> * What is '<TPS admin authentication>', and how does one obtain this? > > I added a reference to the Authentication section in pki(1) and an > explanation that the user must be in the Administrators group. > >> * The 'pki tps-audit-mod --help' command shows the following: >> >> usage: tps-audit-mod [OPTIONS...] >> --action <action> Action: update (default), enable, disable. >> --help Show help options >> --input <file> Input file containing audit configuration. >> --output <file> Output file to store audit configuration. >> >> where the man page does not talk about the 'update (default)' >> switch >> for the '--action <action>' option. >> >> One or more EXAMPLES would be nice, although this will probably suffice >> for the first pass at a man page. > > To clarify this I created a separate description for each usage: > * pki tps-audit-mod --action <action> > * pki tps-audit-mod --input <file path> > > I removed the "update (default)" from the CLI since it can be confusing. Please take a look at the new patch (#623-2). The patch now contains just the new pki-audit man page and line wrapping in the pki man page. The CLI changes to remove the non-working command from non-TPS subsystems have been posted as a separate patch (#636).