[Freeipa-devel] [PATCH] improve cert revocation_reason argument

[Simo Sorce]

On Fri, 2009-05-08 at 15:51 -0400, Rob Crittenden wrote:
> Simo Sorce wrote:
> > On Fri, 2009-05-08 at 14:12 -0400, Rob Crittenden wrote:
> >> Add a min/max range and some documentation on the revocation_reason 
> >> argument. I think it would be a bit much to iterate all the reasons for 
> >> revocation here so I didn't include that.
> > 
> > 
> > Uhmmm using the product, would you be able to pick the right one without
> > having to look at docs ?
> > If not I guess most admins would be tempted to pick one at random ...
> > 
> > Simo.
> > 
> 
> OptionParser formatting leaves a *lot* to be desired. I'm not sure 
> encoding the strings would be any better than forcing them to do a man 
> first. In the UI I think we'll be able to show a textual representation, 
> things are always uglier on the command-line.

In this case I will reluctantly ack :-)

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York