<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Tue, May 3, 2016 at 9:35 AM, Patrice Duc-Jacquet <span dir="ltr"><<a href="mailto:patrice.duc.jacquet@gmail.com" target="_blank">patrice.duc.jacquet@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 05/03/2016 04:41 PM, Rob Crittenden wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Gabe Alford wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
Thank you for your patch as well.<br>
<br>
 >-            doc=_('Reason for revoking the certificate (0-10)'),<br>
 >+            doc=_('Reason for revoking the certificate (0-10). See<br>
RFC 5280 (paragraph 5.3.1) for reason details'),<br>
<br>
Rather than just specifying the RFC with the paragraph to go look up,<br>
can you either add the revocation options or say something like:<br>
<br>
+            doc=_('Reason for revoking the certificate (0-10). See<br>
\'ipa help cert\' for revocation reason details.'),<br>
<br>
IMO, it is a little annoying to go look up revocation reasons when those<br>
reasons can either be added to the help output or exist already in `ipa<br>
help cert`.<br>
</blockquote>
<br>
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.<br>
<br>
rob<br>
<br>
</blockquote></div></div>
Hi everyone<br>
thanks for your valuable comments. I fully agree that it is not recommended to duplicate this information. So as Rob suggested, I should avoid to add this information to cert_revoke option and thus I plan to modify the help message as follow:<br>
<br>
doc=_('Reason for revoking the certificate (0-10). Type "ipa help cert" for reason details'),<br>
<br>
Do you agree with that modification? Thanks in advance and regards<br></blockquote><div><br></div><div>I think the modification is fine. One nitpick that I would have is to say "for revocation reason details."  rather than "for reason details."<br></div><div>Also, don't forget a period after the word "details". :)<br><br></div><div>Gabe<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Pat<br>
</blockquote></div><br></div></div>