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 -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
