[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Pki-devel] [PATCH] pkidaemon man page


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'.

-- Matt

Pki-devel mailing list
Pki-devel redhat com

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]