Re: [Pki-devel] [PATCH] pkidaemon man page
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(a)redhat.com>
To: "pki-devel" <pki-devel(a)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(a)redhat.com
https://www.redhat.com/mailman/listinfo/pki-devel