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

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.