[Pki-devel] [PATCH] 48-2 Fixes for [Patch] 48 Show some more information after installation
Please review the patch with fixes for the review comments for patch 48
On Mon, 2013-04-08 at 13:25 -0500, Endi Sukma Dewata wrote:
--Abhishek
On 4/5/2013 9:59 AM, Abhishek Koneru wrote:
> Some more changes added to the patch. Please ignore the previous post.
> Please review the attached patch.
Some comments:
1. There's a typo:
PKI_CHECK_STATUS_MESSAGE = "COmmand...
2. Please also show the following information:
- Admin username
- Location of client database
- Client certificate nickname
Added these details.
This way the admin knows the parameters needed to use the CLI.
3. Could we move these messages before 'Installation complete'? If the
messages are long it will be more difficult to see the result of the
installation.
Information is printed above the 'Installation complete' message.
4. Some trailing whitespaces.
Fixed.
Attachments:
- pki-akoneru-0048-2-Show-some-more-information-after-installing-a-subsys.patch (text/x-patch — 4.6 KB)