Nice: Conditional ACK Just a couple of minor points to address, if agreed. 1. This line might be too long, could be broken up for better understanding: The pkidaemon command with the 'status' argument provides a way to display the status of all existing PKI instances on a machine, or optionally, an individual PKI instance by specifying an optional [instance-name]. Either more punctuation or maybe two sentences would work nicely. 2. There are three examples of the following or similar: Listing the status of the PKI instance named 'pki-tomcat': pkidaemon status tomcat pki-tomcat These two bold lines talk about an example to come. The first line is the label is the second one. I think it would be clearer if we can get some blank line separation between these two for clarity. That's it. ----- Original Message ----- From: "Matthew Harmsen" <mharmsen@redhat.com> To: "pki-devel" <pki-devel@redhat.com> Sent: Friday, July 17, 2015 7:40:53 PM Subject: [Pki-devel] [PATCH] pkidaemon man page Please review the attached patch which contains a man page for 'pkidaemon'. Thanks, -- Matt _______________________________________________ Pki-devel mailing list Pki-devel@redhat.com https://www.redhat.com/mailman/listinfo/pki-devel