[Freeipa-devel] [PATCH] 0001 provide more information for "ipa cert-revoke -h"

Rob Crittenden rcritten at redhat.com
Tue May 3 14:41:44 UTC 2016


Gabe Alford wrote:
> Hello,
>
> Thank you for your patch as well.
>
>  >-            doc=_('Reason for revoking the certificate (0-10)'),
>  >+            doc=_('Reason for revoking the certificate (0-10). See
> RFC 5280 (paragraph 5.3.1) for reason details'),
>
> Rather than just specifying the RFC with the paragraph to go look up,
> can you either add the revocation options or say something like:
>
> +            doc=_('Reason for revoking the certificate (0-10). See
> \'ipa help cert\' for revocation reason details.'),
>
> IMO, it is a little annoying to go look up revocation reasons when those
> reasons can either be added to the help output or exist already in `ipa
> help cert`.

FTR I added it to the top level help because the reasons are used in 
multiple places and didn't want to duplicate them, and adding them to a 
specific option help would overload it big time IMHO.

rob




More information about the Freeipa-devel mailing list